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] Loading of DICOMs has changed order in 3.8 #4531

Open
salimkanoun opened this issue Nov 19, 2024 · 0 comments
Open

[Bug] Loading of DICOMs has changed order in 3.8 #4531

salimkanoun opened this issue Nov 19, 2024 · 0 comments
Labels
Awaiting Reproduction Can we reproduce the reported bug?

Comments

@salimkanoun
Copy link
Contributor

Describe the Bug

See video for the same case,
The first loading is the current main branche
The second loading is the version before 2.0 conrestone upgrade
https://github.com/user-attachments/assets/add8bfe8-42fb-4e45-bf33-a5854d1a5376

I would be better to keep topToBottum and make it configurable,
The loading process probably needs some rethinking as previous loading feature such as InterleaveTopToButtom are now not accessible any,

Probably we need to define loading strategy :

  • Globally in the app-config
  • Per hanging protocol with customLoadStrategy, if defined it takes priority on the global setting.

The definition of the loading strategy should probably have 2 parameters defined separatly :

  • The loading strategy : Interleave, Nth ....
  • The orientation : Top to bottom, bottom To Top ....

Steps to Reproduce

Load image

The current behavior

bottom to top

The expected behavior

top to bottom

OS

linux

Node version

20

Browser

Chrome latest

@salimkanoun salimkanoun added the Awaiting Reproduction Can we reproduce the reported bug? label Nov 19, 2024
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

1 participant