DICOM browser layout issues

Development
1

I found an issue with the new DICOM browser, and thought that we need a place to discuss these (from my experience there are always a few issues after major changes). Since the PR is closed, we can track the issues on Mantis, but I think we could use discourse for related discussions.

So the one I found is about the browser disappearing after a successful load (here’s the issue). The DICOM browser itself disappears from the layout, which becomes white and the “DICOM database” label shifts to the middle. The “Show DICOM database browser” button remains toggled, but does not have any effect. To make it reappear, need to switch to another layout then press the show button.

I also have a question: if I remember correctly we wanted to keep the popup style browser as an option but I can’t find it in Application Settings. Is it there programmatically just not exposed?

2

It doesn’t seem to happen with all datasets.
Behavior is fine when loading CT/RTDose volumes, but the issue occurs when loading RTStruct.

I’ll compile SlicerRT and debug the problem.

3

Thanks, Kyle! There are no errors (only the usual warnings about geometry and empty structures), so I did not assume it was due to RTSS. I wonder what it can be…

4

I have encountered different DICOM Browser issue recently, not sure if I should report it here or on an issue tracker?

After importing one dataset, series description got multiplied in some series, resulting in destroying the layout of whole browser:
image

Even though the metadata series description seems fine:
image

I can try to upload an anonymized dataset if that helps

5

@JanWitowski I fixed this last week. If you download a recent nightly, then this issue will not occur. You will need to re-import the dataset (or re-update the database if it was from an old one).

1 Like
6

Thanks @cpinter, reloaded the study and it’s fixed!

1 Like
7

The issue with the DICOM browser layout dissapearing should be fixed in this PR: https://github.com/Slicer/Slicer/pull/1200

@cpinter To answer your previous question about the option for the popup style, we decided to only allow the DICOM browser layout rather than complicate things by maintaining both options.

1 Like
8

Interesting. We usually decide to “complicate things” when we introduce a big change like this. Many users will want their popups I’m sure. But I guess we’ll see.

9

In your own module you can still show the widget in a popup. Also, it will be possible to show the DICOM browser view on a secondary screen or floating panel when we implement multi-display layouts.

10

Yes, that will be the proper solution, but realistically that may be a year from now.
I personally like the browser in the layout.