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
Roadmap list of issues/enhancements for Visual DICOM Browser #1162
Comments
@lassoan this should already work, please try to disable DICOM/detailedLogging in the Slicer settings.
yes I agree, this is also described in other points. |
For reference: Error report is addressed in the following PR: |
This enhancement includes an update to CTK and introduces the `ctkDICOMVisualBrowserWidget` to augment DICOM exploration, query, and retrieval capabilities within the DICOM module. The widget is seamlessly integrated into the Slicer DICOM module and is labeled as experimental. For a comprehensive overview and future plans, refer to the roadmap at commontk/CTK#1162. By default, the widget is disabled, and users can access the familiar `ctkDICOMBrowser` when opening the `DICOM` module. To enable the experimental feature, users can toggle the option in the dropdown menu of the `Show DICOM database` pushbutton in the `DICOM` module UI. In `SlicerDICOMBrowser`, this introduces an instance of `ctkDICOMVisualBrowserWidget` alongside the existing `ctkDICOMBrowser` instance. When users activate the experimental feature, widget visibilities are adjusted accordingly. Both widgets share the same DICOM folder set in the DICOM settings. List of CTK changes: ``` $ git shortlog 45f33c81..88ff72b9 --group=author --group=trailer:co-authored-by --no-merges Andras Lasso (1): ENH: Add Visual DICOM Browser (Slicer#1165) Davide Punzo (1): ENH: Add Visual DICOM Browser (Slicer#1165) Jean-Christophe Fillion-Robin (1): ENH: Add Visual DICOM Browser (Slicer#1165) ``` Co-authored-by: Andras Lasso <lasso@queensu.ca> Co-authored-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>
|
This enhancement includes an update to CTK and introduces the `ctkDICOMVisualBrowserWidget` to augment DICOM exploration, query, and retrieval capabilities within the DICOM module. The widget is seamlessly integrated into the Slicer DICOM module and is labeled as experimental. For a comprehensive overview and future plans, refer to the roadmap at commontk/CTK#1162. By default, the widget is disabled, and users can access the familiar `ctkDICOMBrowser` when opening the `DICOM` module. To enable the experimental feature, users can toggle the option in the dropdown menu of the `Show DICOM database` pushbutton in the `DICOM` module UI. In `SlicerDICOMBrowser`, this introduces an instance of `ctkDICOMVisualBrowserWidget` alongside the existing `ctkDICOMBrowser` instance. When users activate the experimental feature, widget visibilities are adjusted accordingly. Both widgets share the same DICOM folder set in the DICOM settings. List of CTK changes: ``` $ git shortlog 45f33c81..88ff72b9 --group=author --group=trailer:co-authored-by --no-merges Andras Lasso (1): ENH: Add Visual DICOM Browser (#1165) Davide Punzo (1): ENH: Add Visual DICOM Browser (#1165) Jean-Christophe Fillion-Robin (1): ENH: Add Visual DICOM Browser (#1165) ``` Co-authored-by: Andras Lasso <lasso@queensu.ca> Co-authored-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>
I have done my last edit. @lassoan it would be nice if you can review (specifically the intial sentence regarding the "patient exploration tool"). |
@lassoan, thanks for the text edits and I have applied yout comments in https://hackmd.io/SFrAGE4CS8uSkI2YvDRKRg?both. I am going to post it now on the Slicer forum @jcfr can you remove or replace the images and videos (see below for the new ones) in #1165 (comment), please? thanks in advance VideoVisuaDICOMBrowser.mp4 |
Is the feature included in today's Slicer Preview Release? If not then please wait. Also, for biggest impact, it may worth making the announcement on Tuesday (many people are not at their computers in the weekend; Monday is often too busy). It would also allow us to do some internal testing with people we ask to have a look (Steve, Rudolf Bumm, ...). |
yes!
ops sorry, I should have waited, I have already posted https://discourse.slicer.org/t/new-feature-visual-dicom-browser/33874 . I can delete it if you think it would be better. |
OK, no problem. There is some time before the weekend, so hopefully it will catch people's attention and they can find some time to try it and give feedback. |
We can also add a follow post on the topic on Tuesday to bring it to the top of the list. Something along those lines: Raw:
Rendered: Now that the feature has been available ✨ for a few day through the Slicer Thanks again for your time 🙏 |
@jcfr @lassoan I was thinking that we may wait for the merge of #1184 (job list) to bring the post up to the list. Let me know. |
for reference: current dev CTK branch current dev Slicer branch |
RoadMap (items are not listed in priority):
long-termENH:
UI customization:
styleSheet
. Probably we should use the styleSheet approach everywhere. For example, the background yellow warning on filters does not work when applying a custom styleSheet. Discuss with Sam for a good design. Ideally the solution should:Issues and minor ENH:
Logic:
framesBatchLimit
variable.Settings:
Visual browser UI:
Patient selection:
Failed to find patient with PatientsName= and PatientID=
This happens when detailed logging in DICOM settings is checked. It is logging printed by DCMTK. Detailed logging is already as default set to false.
Filtering:
Thumbnails && series widgets:
Check also if using a watcher we can automatically do the import.
The local folder should be a "local connection", like a server. So we could add as many we want in the settings and everything will be automatic in the infrastructure (not adding hard coded specil cases, but everything will be independednt), it is just a new "protocol" comunications.
Error report:
ctkDICOMVisualBrowser application:
Reference PR:
The text was updated successfully, but these errors were encountered: