Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Incorrect trailing (FFFE,E00D) Item Delimitation Item in outgoing C-STORE RQs. #109

Closed
gunterze opened this issue Feb 3, 2021 · 1 comment
Closed

Incorrect trailing (FFFE,E00D) Item Delimitation Item in outgoing C-STORE RQs. #109

gunterze opened this issue Feb 3, 2021 · 1 comment
Labels
Type: Bug

Comments

@gunterze
Copy link

gunterze commented Feb 3, 2021

Karnak appends (FFFE,E00D) Item Delimitation Item to encoded data sets in outgoing C-STORE RQs.

E.g.:

$ od --vx karnak-out-image-uncompressed.dcm | tail -n3
2002460 0016 0011 000a 0010 0012 000e 000b 000c
2002500 000c 0010 0011 fffe e00d 0000 0000
2002516

$ od -vx karnak-out-image-compressed.dcm | tail -n3
0033500 989f d9ff fffe e0dd 0000 0000 fffe e00d
0033520 0000 0000
0033524

$ od -vx karnak-out-sr.dcm | tail -n3
0005340 fffe e0dd 0000 0000 fffe e00d 0000 0000
0005360 fffe e0dd 0000 0000 fffe e00d 0000 0000
0005400
@nroduit
Copy link
Collaborator

nroduit commented Feb 7, 2021 •
edited
Loading

Thank you for reporting this problem.
I had put this tag to try to solve another problem where sometimes the DICOM connections do not close (it hangs). It is now removed with this commit.

I have opened a issue regarding the DICOM associations on dcm4che6. Karnak uses the branch image.

@nroduit nroduit closed this as completed Mar 6, 2021
jdcshug added a commit that referenced this issue Aug 16, 2022
@jdcshug
Merge pull request #109 in TELIMA/karnak from feat/update_from_dev to…
af3343a 
… release

* commit '01492377ac83147afd80c38a916f4e32d7c86964':
  feat: remove escape character + add sequence transfer_status + set to 150000 history monitoring
  feat: add catch clauses in order to get the IOExceptions during transfer/transferOther for the monitoring and rethrow them
  fix: unit test because of renaming fwdDescription
  feat: add export functionality for monitoring view
  feat: downgrade liquibase to 4.3.5 because of liquibase/liquibase#2448
  feat: add property in test application.yml
  feat: temporary deactivate
  feat: add automatic cleaning of monitoring history depending on a size limit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gunterze @nroduit