Skip to content

Idc update - #5

Merged
pieper merged 32 commits into
ImagingDataCommons:masterfrom
JamesAPetts:IDC-update
Sep 18, 2020
Merged

Idc update#5
pieper merged 32 commits into
ImagingDataCommons:masterfrom
JamesAPetts:IDC-update

Conversation

@JamesAPetts

@JamesAPetts JamesAPetts commented Sep 17, 2020

Copy link
Copy Markdown

@pieper

  • Update to master..
  • Enable debug extension.
  • Enable DICOM Tag Browser extension with mailTo to support@canceridc.dev.

JamesAPetts and others added 30 commits August 18, 2020 14:01
* feat: 🎸 Update react-vtkjs-viewport usage to use requestPool

* Fix import of react-vtkjs-viewport to cornerstone-tools path.

* Increase maximum load time of MPR test now we are throttling requests.

* Remove debugger

Co-authored-by: Erik Ziegler <erik.sweed@gmail.com>
 - @ohif/extension-cornerstone@2.9.0
 - @ohif/extension-dicom-rt@0.4.0
 - @ohif/extension-dicom-segmentation@0.4.0
 - @ohif/extension-vtk@1.7.0
 - @ohif/core@2.10.0
 - @ohif/viewer@4.4.0
…t requests (OHIF#2000)

Co-authored-by: Danny Brown <danny.ri.brown@gmail.com>
 - @ohif/extension-cornerstone@2.9.1
 - @ohif/viewer@4.4.1
improve filter/promote to be applied on multiple series instances

✅ Closes: 1532

Co-authored-by: Rodolfo Ladeira <rodolfo_radical@hotmail.com>
Co-authored-by: Danny Brown <danny.ri.brown@gmail.com>
 - @ohif/extension-cornerstone@2.9.2
 - @ohif/viewer@4.5.1
* feat: 🎸 Update react-vtkjs-viewport usage to use requestPool

* Fix import of react-vtkjs-viewport to cornerstone-tools path.

* Increase maximum load time of MPR test now we are throttling requests.

* fix: 🐛 Fail gracefully on an MPR load error

* Respond to reviewer comments.
 - @ohif/extension-vtk@1.7.1
 - @ohif/viewer@4.5.2
…umber by date/time. (OHIF#2010)

* Sort based on SeriesNumber and SeriesDate/SeriesTime.

* Harden, and perform final sort in algorithm if last N entries have the same SeriesNumber.

* Switch to insertion rather than sorting as sorting is too slow. Reimplement low priority sorting into new insertion method.

* Fix local file viewing.
 - @ohif/extension-dicom-html@1.2.5
 - @ohif/extension-dicom-microscopy@0.51.1
 - @ohif/extension-dicom-pdf@1.0.4
 - @ohif/extension-dicom-rt@0.4.1
 - @ohif/extension-dicom-segmentation@0.4.1
 - @ohif/extension-vtk@1.7.2
 - @ohif/core@2.10.1
 - @ohif/viewer@4.5.3
* fix: 🐛 Upgrade dcmjs version to support more SEGs
 - @ohif/extension-cornerstone@2.9.3
 - @ohif/extension-dicom-html@1.2.6
 - @ohif/extension-dicom-rt@0.4.2
 - @ohif/extension-dicom-segmentation@0.4.2
 - @ohif/extension-vtk@1.7.3
 - @ohif/core@2.10.2
 - @ohif/viewer@4.5.4
* WIP debug dialog

* Rename the p10 downloader extension to debugger extension, add button to toolbar. Deactivate it by default.

* Fix unit tests
 - @ohif/extension-cornerstone@2.9.4
 - @ohif/extension-debugging@0.1.3
 - @ohif/extension-dicom-html@1.2.7
 - @ohif/extension-dicom-microscopy@0.51.2
 - @ohif/extension-dicom-pdf@1.0.5
 - @ohif/extension-dicom-rt@0.4.3
 - @ohif/extension-dicom-segmentation@0.4.3
 - @ohif/extension-lesion-tracker@0.2.1
 - @ohif/extension-vtk@1.7.4
 - @ohif/core@2.10.3
 - @ohif/viewer@4.5.5
* Multiple series search for google cloud adapter.

* Revert IDC config.

* fix: 🐛 Series filtering on multiple series for google

* Revert changes to default config.

* Address reviewers comments.

* Fixed spelling mistakes
 - @ohif/extension-vtk@1.7.5
 - @ohif/core@2.10.4
 - @ohif/viewer@4.5.6
…HIF#2034)

* Highlight for RTSTRUCT.

* SEG temp crosshairs.

* Disable RT highlighting for now.

* Remove TODO
 - @ohif/extension-debugging@0.1.4
 - @ohif/extension-dicom-rt@0.4.4
 - @ohif/extension-dicom-segmentation@0.4.4
 - @ohif/viewer@4.5.8
* WIP render top level tags.

* Add drop down to select series.

* Fix errors with type 2 sequences.

* WIP swap instance.

* Fix formatting and make fullscreen.

* Remove debuggers.

* Finish formatting.

* Fix error with double SeriesNumber deconstruction.

* Address reviewer comments.
 - @ohif/extension-dicom-html@1.2.8
 - @ohif/extension-dicom-microscopy@0.51.3
 - @ohif/extension-dicom-pdf@1.0.6
 - @ohif/extension-dicom-rt@0.4.5
 - @ohif/extension-dicom-segmentation@0.4.5
 - @ohif/extension-dicom-tag-browser@0.0.2
 - @ohif/extension-vtk@1.7.6
 - @ohif/core@2.10.5
 - @ohif/ui@1.5.4
 - @ohif/viewer@4.5.9
* WIP debug dialog

* Rename the p10 downloader extension to debugger extension, add button to toolbar. Deactivate it by default.

* Fix unit tests

* WIP

* WIP

* Finish mailTo
 - @ohif/extension-debugging@0.1.5
 - @ohif/extension-vtk@1.7.7
 - @ohif/ui@1.5.5
 - @ohif/viewer@4.5.10
…#2044)

* WIP debug dialog

* Rename the p10 downloader extension to debugger extension, add button to toolbar. Deactivate it by default.

* Fix unit tests

* WIP

* WIP

* Finish mailTo

* tooltop
@pieper

pieper commented Sep 17, 2020

Copy link
Copy Markdown
Member

LGTM 👍

@wlongabaugh @fedorov I'm updating the sandbox with this version and the configs to turn off measurements and allow multiple url parameters.

@pieper

pieper commented Sep 17, 2020

Copy link
Copy Markdown
Member

Okay, sandbox update complete. You can test there and merge this if you like what you see.

@fedorov

fedorov commented Sep 18, 2020

Copy link
Copy Markdown
Member

I left a comment in OHIF#1532 (comment) - multiple series doesn't work for me, or I don't know how to use it, but let's go ahead with the update anyway.

@fedorov

fedorov commented Sep 18, 2020

Copy link
Copy Markdown
Member

@pieper I know it's a silly question, but can we double-check sandbox was indeed updated?

From the comments above, it seems that we should have the following options:

  • debug extension
  • DICOM tag browser extension
  • hover tooltips for the segments/rtstructs
  • mailto extension
  • split query parameters to support series selection

and I don't see any of the above. could it be a wrong config is used or something?

@pieper
pieper merged commit c279815 into ImagingDataCommons:master Sep 18, 2020
@JamesAPetts
JamesAPetts deleted the IDC-update branch October 7, 2020 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants