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.

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

Invalid tag in sequence : Unable to parse MR Dicom file in dcmjs. Cornerstone able to parse and works fine #363

Open
subhasishdash opened this issue Jul 31, 2023 · 1 comment

Comments

@subhasishdash
Copy link

subhasishdash commented Jul 31, 2023

Hi dcmjs team,
I am using OHIF viewer and it uses dcmjs to parse DICOM files.
The attached MR study fails to parse with dcmjs, but it works fine with cornerstone parser Cornerstone file loader

If possible please add a file loader page like cornerstone as it's easy and quick to check/test.

907375E7.zip

Please let me know how can I fix this.

I tried the following:

  • Regenerate dictonary.js from dcmjs, didn't work.
  • Tried to add a if condition to getNextSequenceItemData to skip if the var nextTag = Tag.readTag(stream); is 2001, didn't work.

I also saw this fix for dcmtk, not sure if it's related.

@subhasishdash subhasishdash changed the title Invalid tag in sequence : Unable to parse MR Dicom file. Not sure why it's happening. Invalid tag in sequence : Unable to parse MR Dicom file in dcmjs. Cornerstone able to parse and works fine Jul 31, 2023
@pieper
Copy link
Collaborator

pieper commented Aug 31, 2023

Hi - thanks for the report and for sharing the data. Can you post the error message you get?

Would you be okay with this data being added to https://github.com/dcmjs-org/data ? If I do that would you make a PR to add a test to use for debugging? (okay if it fails at first).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants