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] DICOM Microscopy worker is called without the Public URL #4906

Open
salimkanoun opened this issue Mar 31, 2025 · 3 comments
Open

[Bug] DICOM Microscopy worker is called without the Public URL #4906

salimkanoun opened this issue Mar 31, 2025 · 3 comments
Labels
Awaiting Reproduction Can we reproduce the reported bug?

Comments

@salimkanoun
Copy link
Contributor

Describe the Bug

Thanks to @wayfarer3130 PR #4891 the dicom-microscopy-viewer now can be retrieved if bundled with a public URL with this additional fix : https://github.com/OHIF/Viewers/pull/4902/files

However there is another js assets that is missing the PUBLIC_URL, which is missing PUBLIC_URL : /dicom-microscopy-viewer/index.worker.min.worker.js

It seems to from dicom-microscopy-viewer library itself.

For now the layout of dicom-microscopy-viewer starts but remains empty as the worker cannot be loaded

Steps to Reproduce

  1. Use a build with public url
  2. Open the microscopy mode
  3. images doesn't load

The current behavior

no loading of wsi

The expected behavior

Loading of WSI should work even with public url

OS

Linux

Node version

20

Browser

Chrome

@salimkanoun salimkanoun added the Awaiting Reproduction Can we reproduce the reported bug? label Mar 31, 2025
@wayfarer3130
Copy link
Contributor

This is an issue in the dicom-microscopy-viewer library needing to have it's files served as /dicom-microscopy-viewer/ There isn't a good fix for it in OHIF other than mounting the files under the path that is expected. It might be possible to fix the included library as an external fix by changing to a relative dependency, but that might not work effectively either as it requires the including library (OHIF in this case) to use the right include setup path, or ensure that the wasm include paths are setup correctly to be relative to the load path for the dicom-microscopy-viewer.

@wayfarer3130
Copy link
Contributor

@salimkanoun - feel free to create PR for dicom-microscopy-viewer to update the build paths as suggested. That can then be included there and then the version of the microscopy library updated in OHIF

@sedghi
Copy link
Member

sedghi commented Apr 1, 2025

We can't really blame dicom-microscopy-viewer. If OHIF v3 has worked well with dicom-microscopy on the public URL for the past three years, it should continue to do so. We're not sure how many OHIF users share the same use case as reported in this issue. I tried to fix dicom-microscopy-viewer, but I wasn't successful.

ImagingDataCommons/dicom-microscopy-viewer#177

I believe it probably has something to do with how webpack worker is used in the UMD.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Reproduction Can we reproduce the reported bug?
Projects
None yet
Development

No branches or pull requests

3 participants