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

Improved user experience while handling fractional SEG #1369

Closed
fedorov opened this issue Jan 13, 2020 · 3 comments
Closed

Improved user experience while handling fractional SEG #1369

fedorov opened this issue Jan 13, 2020 · 3 comments
Labels
IDC:candidate Possible feature requests for discussion, and if we agree, they can be relabeled IDC:priority

Comments

@fedorov
Copy link
Member

fedorov commented Jan 13, 2020

Before Creating an issue

  • Are you running the latest version?

No, this is tested with 3.3.7.

  • Are you reporting to the correct repository?

I think so.

  • Did you search existing issues?

Yes, the closest related issue is this #1346, but the present issue not about support of fractional segmentation, but about another (hopefully, easier) issue about reporting to the user that fractional segmentation is currently not supported.

Bug Report

Dragging SEG series 52001 "Breast Tissue Segmentation" into the viewer does not result in segmentation rendering, and does not produce any hints to the user as to why.

The same series shows up in the latest stable version of Slicer 4.10.2 with the Quantitative Reporting extension installed as shown below:

image

Describe the Bug

Segmentation series does not show in the viewer, and it is not clear why.

What steps can we follow to reproduce the bug?

  1. Open this link: https://idc-sandbox-000.firebaseapp.com/projects/chc-tcia/locations/us-central1/datasets/ispy1/dicomStores/ispy1/study/1.3.6.1.4.1.14519.5.2.1.7695.1700.301904873570071685223271646148
  2. Drag series 102001 into the viewer - nothing happens. Looking at the developer console, there is the error below:

image

Suggestion for improvement of the functionality

I suggest it would be helpful if there was some more clear way of communicating to the user if certain functionality is not supported. In this case, even from the developer console it is not clear if fractional segmentations are supported or not (but it is alluded to in another thread that they are not #1346).

After completing this issue report, I realized that this is observed in 3.3.7, which is what is used on IDC sandbox, and not the latest 3.3.8. It might be that the latest version behaves differently.

@fedorov fedorov added Community: Report 🐛 IDC:candidate Possible feature requests for discussion, and if we agree, they can be relabeled IDC:priority labels Jan 13, 2020
@fedorov
Copy link
Member Author

fedorov commented Jan 13, 2020

@pieper updated the IDC sandbox, and I confirm the issue remains in 3.3.8.

@fedorov
Copy link
Member Author

fedorov commented Jan 17, 2020

Based on some discussions about this, one idea could be the following:

  1. show every single series in the left column of the interface (including SEG etc)
  2. when study is received, identify series that are not supported
  3. mark series that are not supported with an exclamation mark or something like that

@Punzo
Copy link
Contributor

Punzo commented Sep 13, 2022

this has been implemented in the context of series warning UI

@Punzo Punzo closed this as completed Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IDC:candidate Possible feature requests for discussion, and if we agree, they can be relabeled IDC:priority
Projects
None yet
Development

No branches or pull requests

2 participants