-
Notifications
You must be signed in to change notification settings - Fork 113
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.
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
Order of tags gets messed up when edition DICOM SEG #365
Comments
Sample original and edited files |
@pieper any ideas? |
Hi @emelalkim thanks for your patience - I think the issue is that In dcmjs that's stored in the |
Hi @pieper, thank you for the feedback
Am I doing it wrong? |
Hmm, either 2.25.xxx or 1.3.xxx should be fine. There are rules in the standard about what's allowed in these strings of numbers but software like dcmjs shouldn't care. (We use the 2.25 form since it's considered to be the correct way to prefix a random number, while the 1.xxx forms are related to registered prefixes). In addition to
Maybe you need to add
? If it's still a problem the code to write the data shouldn't be too hard to trace through. |
hi @pieper,
I was able to resolve my problem with adding
I don't know which part in the library needs to be fixed if any, and I don't have time to work on it right now but if you give me pointers I will try to work on it later |
I'm glad you found a good solution @emelalkim. I agree it's not clear if this requires a change in the library. |
Originally posted by @pieper in #64 (comment)
Hi, We are using this method to create copies of DICOM objects with different UIDs.
The MRIs are OK, but the segmentations throws this when I run dciodvfy
Error - Tags out of order - trailing garbage, wrong transfer syntax, or not valid DICOM
Error - MediaStorageSOPInstanceUID but missing SOPInstanceUID and not a directory
Error - MediaStorageSOPClassUID but missing SOPClassUID and not a directory
I was able to reproduce it locally using this code
The text was updated successfully, but these errors were encountered: