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

Idc update #8

Merged
merged 42 commits into from
Oct 7, 2020
Merged

Idc update #8

merged 42 commits into from
Oct 7, 2020

Conversation

JamesAPetts
Copy link

Updating to include recent tickets from master: Viewers/ 2003, 2059, 2064.

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 <[email protected]>
improve filter/promote to be applied on multiple series instances

✅ Closes: 1532

Co-authored-by: Rodolfo Ladeira <[email protected]>
* 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.
…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.
* fix: 🐛 Upgrade dcmjs version to support more SEGs
* WIP debug dialog

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

* Fix unit tests
* 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
…HIF#2034)

* Highlight for RTSTRUCT.

* SEG temp crosshairs.

* Disable RT highlighting for now.

* Remove TODO
* 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.
* 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
JamesAPetts and others added 12 commits September 17, 2020 20:09
…#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
…ent after space. (OHIF#2053)

* Sort tag browser, add items, add indent.

* Remove debugger.
…2062)

* Update react-cornerstone-viewport

* Bump react-cornerstone-viewport version.

* Comment out tests which always cause problems.
Change start_url in manifest.json from "./index.html" to "./"

Co-authored-by: Woonchan Cho <[email protected]>
…tructuredReport (OHIF#2038)

fixes OHIF#1714

Call getInstanceCount() instead of manually checking length (which does not work)

Co-authored-by: Stefan Silviu <[email protected]>
@pieper pieper merged commit 75b5d9d into ImagingDataCommons:master Oct 7, 2020
@pieper
Copy link
Member

pieper commented Oct 7, 2020

@JamesAPetts
Copy link
Author

Oops, sorry about this! So I tested but I still had dcmjs linked. The merge conflicts reset the dcmjs version to the old version for the deploy.

Unlinked and updated PR inc

Screenshot 2020-10-08 at 09 59 36

@JamesAPetts JamesAPetts mentioned this pull request Oct 8, 2020
@JamesAPetts
Copy link
Author

Fixed here: #9

@JamesAPetts JamesAPetts deleted the IDC-update branch October 8, 2020 09:04
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.

9 participants