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

Reading dicom file #373

Open
Hyunsoo-dev opened this issue Dec 11, 2023 · 2 comments
Open

Reading dicom file #373

Hyunsoo-dev opened this issue Dec 11, 2023 · 2 comments

Comments

@Hyunsoo-dev
Copy link

Hyunsoo-dev commented Dec 11, 2023

When i used the code below,

let DicomDict = dcmjs.data.DicomMessage.readFile(dicomFile);

i �got the error as below.

Using multiple character sets is not supported: ,ISO 2022 IR 149

But the standard document says you can put multiple values in a specific character set (0008, 0005).
I'm curious why you limited the number of character set, can you change this restriction?
image

/// dcmjs.js -> line 10072
if (readInfo.values.length > 1) {
    if (ignoreErrors) {
	    log.warn("Using multiple character sets is not supported, proceeding with just the first character set", readInfo.values);
    } else {
	    throw Error("Using multiple character sets is not supported: ".concat(readInfo.values));
   }
}
readInfo.values = ["ISO_IR 192"]; // change SpecificCharacterSet to UTF-8
@pieper
Copy link
Collaborator

pieper commented Dec 11, 2023

There were probably no example datasets available when the code was written or it wasn't a priority for any of the developers/users. It would be great if you could propose a change to make dcmjs more compliant with the standards and provide tests so that further compliance can be tested. It would be good to compare how other toolkits work with this kind of data.

@WillianVarela
Copy link

WillianVarela commented Jun 18, 2024

setting the option { ignoreErrors: true } worked for me

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

3 participants