-
Notifications
You must be signed in to change notification settings - Fork 266
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
Web Worker unable to load image with JP2 Header file. #238
Comments
Just for reference, the file This is clearly unacceptable from a strict point of view:
The question left open is of course: what is the course of action for cornerstone: should support or not ? |
Is it possible to strip this header and 'normalise' the image in javascript before trying to load it in cornerstone? |
AFAIK the codec used by cornerstone for JPEG2000 is OpenJPEG. Since the above JP2 is well handled by OpenJPEG, I would suggest to update the support for JPEG2000 to handle both J2K and JP2 and be done with this (this is what GDCM does BTW). OpenJPEG has always had a poor support for parsing operation (non-decoding ones). For instance: so I believe this is a no-go solution (unless someone wants to implement a simple JP2 parser). For reference: |
@chafey You mentionned in some other bug report that focus should be on legal image only. This one has a JP2 header which has been prohibited since day 1 in the DICOM standard. |
When loading the Web Worker the following image throws an error.
https://sourceforge.net/p/gdcm/gdcmdata/ci/master/tree/ELSCINT1_JP2vsJ2K.dcm?format=raw
There are a number of images of this type which throws the error.
The loaded image does not have a getCanvas() method attached to it.
The text was updated successfully, but these errors were encountered: