-
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
Write issue for JPEG2000Loseless images #375
Comments
Probably only a few transfer syntaxes were tested originally. I'm sure it would be possible to add support for that, but historically codecs haven't been easily dealt with. This is probably changing and it would be great if someone could investigate using dicom-codecs-wasmjs with dcmjs. For anonymizing did you look at the existing code? https://github.com/dcmjs-org/dcmjs/blob/master/src/anonymizer.js https://github.com/dcmjs-org/dcmjs/blob/master/test/anonymizer.test.js |
Sure will take a look into that, thanks. |
I'm not sure but I believe only uncompressed pixel data is currently supported. |
We are trying to anonymize a DICOM file which is a JPEG2000Loseless type (Transfer Syntax ID - 1.2.840.10008.1.2.4.90)
But when we try to perform the .write() operation, we are getting an error:
First argument to DataView constructor must be an ArrayBuffer
But it's working fine for other DICOM file types.
We tried opening this file in MicroDicom Viewer and it's loading up fine.
Can anyone help understand why we might be getting this error ?
The steps we are using are as follows:
We are generating the buffer for the file and storing it in a variable named buffer.
Which also brings me to the next question, is this the proper way to anonymize or should we be using the
dicomDict.upsertTag()
function ?The text was updated successfully, but these errors were encountered: