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

[Bug] DicomJSONDatasource volume loading problem #3859

Closed
lambacini opened this issue Dec 16, 2023 · 14 comments
Closed

[Bug] DicomJSONDatasource volume loading problem #3859

lambacini opened this issue Dec 16, 2023 · 14 comments
Assignees
Labels
fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org)

Comments

@lambacini
Copy link

lambacini commented Dec 16, 2023

Describe the Bug

After CS3D Version update, volume loading with dicomjson datasource is broken.
There doesn't seem to be any problem when the viewport is stack, but there is a problem if we want to load it as a volume.

Steps to Reproduce

( no changes were made to the code base for testing. active master branch was used. )

Loading as Stack (Works Smoothly)
http://localhost:3000/viewer/dicomjson?url=https://ohif-dicom-json-example.s3.amazonaws.com/LIDC-IDRI-0001.json

Loading as Volume
http://localhost:3000/viewer/dicomjson?url=https://ohif-dicom-json-example.s3.amazonaws.com/LIDC-IDRI-0001.json&hangingprotocolid=mpr

image

The current behavior

.

The expected behavior

.

OS

Ubuntu 22.04

Node version

18.18.0

Browser

120.0.6099.71 (x64)

@lambacini
Copy link
Author

Ohif 3.7 works fine. After updating cs3d ( ohif 3.8 & cs3d 1.41.0 ) it is not working correctly.
You can reproduce on the master branch.

@lambacini
Copy link
Author

From what I can discern, the way the problem occurs ;
in the cornerstone3D library, BaseStreamingImageVolume.ts calls (via imageLoader) the wadouri.loadImage method, when imageRetrieveConfiguration is not set, options.loader is assigned "this" i.e. baseStreamingImageVolume.

in the wadouri.loadImage method, if options.loader is present, this loader is called , but the prefix is removed because it is invoked with the parsedImageId.url parameter (not xhrRequest). Consequently, the image loader cannot be located when the loadImageFromImageLoader method is called, as the prefixes are stripped.

image

It may not be the right solution, but it temporarily solved my problem (I needed an urgent solution :) )

image

I hope I have understood and explained it correctly.

@sedghi

@ahlaughland
Copy link

I have the same problem as I'm sure anyone who updates will too. It happens with the cornerstone dependency updates to streaming-image-volume-loader at commit 9d1884d

I rolled back to "@cornerstonejs/streaming-image-volume-loader": "^1.27.3", for now.

Hope this helps.

@jianyaoo
Copy link

jianyaoo commented Jan 2, 2024

When the 3d version was upgraded to 1.42.0, I encountered the same error problem when dicom loaded wadouri

@Ncp-Zg
Copy link

Ncp-Zg commented Jan 9, 2024

I had the same problem after the update.
Is there any change ?

@AmrGablla
Copy link

I have the same error when i use the cornerstone 3D 1.43.7 versions.

@sedghi
Copy link
Member

sedghi commented Jan 9, 2024

Can you share reproducible steps?

@lambacini
Copy link
Author

@sedghi
Copy link
Member

sedghi commented Jan 9, 2024

I see and confirm, will look

@sedghi sedghi added Bug Verified Bug reported, reproducible, and verified. Awaiting Reproduction Can we reproduce the reported bug? and removed Awaiting Reproduction Can we reproduce the reported bug? labels Jan 9, 2024
@sedghi sedghi self-assigned this Jan 9, 2024
@sedghi
Copy link
Member

sedghi commented Jan 9, 2024

Just an update: this was added after HTJ2K additions to cs3d, still investigating

@wayfarer3130
Copy link
Contributor

This is resolved using your solution in PR cornerstonejs/cornerstone3D#990

@sedghi
Copy link
Member

sedghi commented Jan 11, 2024

You can update your cornerstone versions and it is fixed now, until I push a PR to fix upstream. Meanwhile change the loading strategy for MPR and this will fix your missing slices until we properly fix that in this issue #3890

@sedghi sedghi reopened this Jan 11, 2024
@lambacini
Copy link
Author

Thank you for your interest, i will test it and get back to you.

@sedghi sedghi added Bug Verified Bug reported, reproducible, and verified. fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org) and removed Bug Verified Bug reported, reproducible, and verified. labels Jan 12, 2024
@sedghi
Copy link
Member

sedghi commented May 1, 2024

We just release the OHIF 3.8, you can find more details here https://ohif.org/release-notes/3p8/
If you still encounter this issue in 3.8, please re-open this.

@sedghi sedghi closed this as completed May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org)
Projects
None yet
Development

No branches or pull requests

7 participants