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

More graceful handling of unsupported DICOM modalities is needed #492

Open
fedorov opened this issue Nov 8, 2023 · 1 comment
Open

More graceful handling of unsupported DICOM modalities is needed #492

fedorov opened this issue Nov 8, 2023 · 1 comment

Comments

@fedorov
Copy link

fedorov commented Nov 8, 2023

If a study contains a Structured Report instance, it will manifest itself in the error shown in #491. It would be a lot more helpful if instead it was communicated to the user that SR modality is not supported.

Also, it is not clear if after that error the viewer is expected to be usable. After dismissing the error notification, I tried to load the CT series, but that failed. If I reload the page, I will get the same error (since the SR instance in the bucket is still there), so this becomes a blocker.

@floryst
Copy link
Collaborator

floryst commented Nov 8, 2023

Semi-duplicate of #486

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