fix(viewports): The display of linked viewports during drag and drop has a race #3286

wayfarer3130 · 2023-03-28T18:22:53Z

Context

If multiple react state events occur near each other, react can decide to combine this, or can serialize them, or any combination thereof. As well, any number of renderings or none can occur in between these. This combination can lead to viewports
being in the midst of being created or destroyed when being rendered, or a partial new rendering occur, causing exceptions.

Changes & Results

There are two parts to the fix. First, not reusing viewport Id's at all means that create/destroy events on single viewport id's don't occur. Second, making the drag and drop and segmentation updates atomic (single operation done at once) means it occurs consistently and in the same order and doesn't invoke multiple renders. This also, incidentally, makes it faster.

Testing

To test, the easiest way is to open up the test:e2e:serve environment, and view the SEG containing series in basic test mode
Try it with various layouts and protocol stages with:
Seg in one viewport, primary images in one or two other viewports
Ensure that you can load the seg, and that after you can:

Add the SEG to a blank viewport
Navigate each image series separately
Change layouts still
Note that during changing layouts there are some issues with remembering the positioning of multiple viewports containing the same series.

Checklist

PR

[] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[] My code has been well-documented (function documentation, inline comments,
etc.)

Public Documentation Updates

[] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[] "OS:
[] "Node version:
[] "Browser:

netlify · 2023-03-28T18:23:01Z

✅ Deploy Preview for ohif-platform-docs canceled.

Name	Link
🔨 Latest commit	`47fe475`
🔍 Latest deploy log	https://app.netlify.com/sites/ohif-platform-docs/deploys/6426e46fdc8fbb00082d2ac8

netlify · 2023-03-28T18:23:06Z

✅ Deploy Preview for ohif-platform-viewer ready!

Name	Link
🔨 Latest commit
🔍 Latest deploy log	https://app.netlify.com/sites/ohif-platform-viewer/deploys/6426fde2eabf0203d45e96cd
😎 Deploy Preview	https://deploy-preview-3286--ohif-platform-viewer.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

extensions/cornerstone-dicom-seg/src/utils/_hydrateSEG.ts

extensions/cornerstone/src/services/ViewportService/CornerstoneViewportService.ts

wayfarer3130 · 2023-03-28T18:25:26Z

extensions/cornerstone/src/services/ViewportService/CornerstoneViewportService.ts

@@ -624,6 +629,10 @@ class CornerstoneViewportService extends PubSubService

    const viewportInfo = this.getViewportInfo(viewport.id);

+    if (!viewportInfo) {


This doesn't seem to happen any longer as it was a race condition causing rendering to fail multiple times.

wayfarer3130 · 2023-03-28T18:26:45Z

platform/ui/src/contextProviders/ViewportGridProvider.tsx

@@ -34,6 +34,16 @@ const DEFAULT_STATE = {

 export const ViewportGridContext = createContext(DEFAULT_STATE);

+const isReuseableViewport = (oldViewport, newViewport) => {


This code isn't strictly necessary, but it does reuse viewports when they are the same size/display sets, and are immediately redisplayed, and otherwise gets a new viewport id every time.

platform/ui/src/contextProviders/ViewportGridProvider.tsx

codecov · 2023-03-28T18:29:21Z

Codecov Report

Merging #3286 (47fe475) into v3-stable (9f1e813) will increase coverage by 12.64%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           v3-stable    #3286       +/-   ##
==============================================
+ Coverage      25.15%   37.80%   +12.64%     
==============================================
  Files            119       85       -34     
  Lines           2862     1386     -1476     
  Branches         555      305      -250     
==============================================
- Hits             720      524      -196     
+ Misses          1856      695     -1161     
+ Partials         286      167      -119

see 76 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ca3b83b...47fe475. Read the comment docs.

cypress · 2023-03-28T18:32:29Z

Passing run #3082 ↗︎

0	35	0	0	0

Details:

fix: Exception thrown on change displayset after double click
Project: Viewers	Commit: `47fe4753e1`
Status: Passed	Duration: 03:13 💡
Started: Mar 31, 2023 1:52 PM	Ended: Mar 31, 2023 1:56 PM

_{This comment has been generated by cypress-bot as a result of this
project's GitHub integration settings.}

platform/core/src/services/ViewportGridService/ViewportGridService.ts

platform/ui/src/contextProviders/ViewportGridProvider.tsx

sedghi

please see my comments

platform/ui/src/contextProviders/ViewportGridProvider.tsx

ladeirarodolfo · 2023-03-28T22:27:15Z

platform/core/src/services/ViewportGridService/ViewportGridService.ts

@@ -38,9 +37,6 @@ class ViewportGridService extends PubSubService {
    if (setActiveViewportIndexImplementation) {
      this.serviceImplementation._setActiveViewportIndex = setActiveViewportIndexImplementation;
    }
-    if (setDisplaySetsForViewportImplementation) {
-      this.serviceImplementation._setDisplaySetsForViewport = setDisplaySetsForViewportImplementation;


For sake of backward compatibility I would say to not remove this. Instead we just factory to the new reducer you have created.

Also, if we decide to really change this we need to ensure the usage of it is also updated (cornerstone-dicom-seg, cornerstone-dicom-sr)

It is backwards compatible - I added a function that supports that directly, but I'm really against having two functions to call in the back end as it makes it harder to figure out what is happening. We are still pre-release on v3-stable so don't have to absolutely worry about backwards compatibility.

jbocce · 2023-03-29T10:58:44Z

Hi @wayfarer3130 , thanks for requesting a review. Please update the PR with details on how to test it. Thanks so much.

jbocce

Looks good. Just some comments to address.

extensions/cornerstone/src/services/ViewportService/CornerstoneViewportService.ts

platform/ui/src/contextProviders/ViewportGridProvider.tsx

wayfarer3130 · 2023-03-30T12:29:58Z

extensions/cornerstone/src/getHangingProtocolModule.ts

@@ -303,11 +303,11 @@ const mprAnd3DVolumeViewport = {
 function getHangingProtocolModule() {
  return [
    {
-      id: 'mpr',
+      name: 'mpr',


Using id for the hanging protocol conflicts with the id provided by the extension manager, and that causes the id to be updated and sometimes fails displaying the protocol stages.

wayfarer3130 · 2023-03-30T12:30:47Z

platform/core/src/extensions/ExtensionManager.ts

@@ -283,6 +283,11 @@ export default class ExtensionManager {
            // Default for most extension points,
            // Just adds each entry ready for consumption by mode.
            extensionModule.forEach(element => {
+              if (!element.name) {


Not defining the name causes loading issues in various places, so it is worth having a check here on load.

extensions/cornerstone/src/services/ViewportService/CornerstoneViewportService.ts

jbocce · 2023-03-30T18:40:57Z

platform/ui/src/contextProviders/ViewportGridProvider.tsx

+
+          const displaySetOptions = updatedViewport.displaySetOptions || [];
+          if (!displaySetOptions.length) {
+            // Copy all the display set options, assuming a full set of displa


typo at very end of line "display"

jbocce

The commits since my last review look good.

I did find one issue that works on demo but not on this PR's deploy preview. Here are the steps...

Launch a series into MPR.
Double click one of the viewports to one up.
Swap in a different reconstructable series.
The viewer goes black with the following exception in the dev tools console...

react_devtools_backend.js:2655 TypeError: Cannot read properties of undefined (reading 'viewportIndex')
    at ViewportGridProvider.tsx:195:36
    at Array.forEach (<anonymous>)
    at ViewportGridProvider.tsx:190:19
    at Object.co [as useReducer] (react-dom.production.min.js:159:447)
    at t.useReducer (react.production.min.js:23:173)
    at yn (ViewportGridProvider.tsx:325:41)
    at io (react-dom.production.min.js:157:137)
    at jo (react-dom.production.min.js:180:154)
    at Ys (react-dom.production.min.js:269:343)
    at Ol (react-dom.production.min.js:250:347)

* fix(volumeLoad): should not have missing slices when loading (#3287) * fix(volumeLoad): should not have missing slices when loading * add review comments * feat(DoubleClick): double click a viewport to one up and back (#3285) * feat(DoubleClick): double click a viewport to one up and back Added a toggleOneUp command that puts the active viewport into a 1x1 grid layout and it toggles out of 'one-up' by restoring its saved 'toggleOneUpViewportGridStore' from the StateSyncService. Added double click customization for the Cornerstone extension with the default double click handling being the toggleOneUp command. Added a cypress test for the double click functionality. * PR feedback: - tracked viewport measurements no longer show as dashed when toggling one up - disallowed double clicking near a measurement - updated cornerstone3D dependencies to fix double click of TMTV and volume viewport 3D - created ViewportGridService.getLayoutOptionsFromState * Updated the ViewportGridService docs. * Switched to using 'cornerstoneViewportClickCommands' and consistency with the context menu clicks. * feat(tmtv): add more stages to pt/ct (#3290) * feat(tmtv): add more stages to pt/ct * make error stage change to info * apply review comments * fix(viewports): The display of linked viewports during drag and drop has a race (#3286) * fix: The display of linked viewports during drag and drop has a race * PR review comments * fix: Segmentation display two up * Removing console logs * Fix the blank viewport can have stuff added to it * Fix the null name on HP module * Fix the navigate to initial image * Fix the nth interleave loader * Fix the unit tests * PR comments - docs mostly * fix: Exception thrown on change displayset after double click * feat(multiframe): enhanced support for multiframe dicom (#3164) * Changes in cswil version and multiframe * Minor changes * wip * Adding support for NM multiframe images * Applying PR suggestions * fixing package versions * Restoring default.js config file * Check if NM subtype is reconstructable * Restore default.js values * refactore code * feat: add flag for strict zspacing --------- Co-authored-by: Alireza <[email protected]> * feat(URL): add param for initial series and sop uids to display (#3265) * feat: Allow navigating to a specified series and sop instance This was a feature in OHIF v2, so adding it to v3, albeit with new parameters. feat: Allow comma separated as well as repeated args params * docs * Test fixes * feat: Navigate to SOP selected - PR fixes * Updated docs * PR fixes * fix(crosshairs): suppressed error for crosshair (#3237) * fix(SRTool): Ellipse Display for DICOMSR Tool (#3307) * feat(App): support async config function (#3313) * fix(URL): allow multi filter query (#3314) * fix(viewport): Initial blank image on SR/SEG initial display (#3304) * fix: Blank display area on initial DICOM SR load * Docs * Fix a NPE * fix(ROIThreshold): fix setWeight not updating properly for ROIThreshold panel (#3315) * fix(hp): Add displaySetMatchDetails and displaySets to options in (#3320) The sameAs function requires displaySetMatchDetails and displaySets to compare attributes between display sets, but these properties were not being passed into the options object. This commit adds the necessary lines of code to include displaySetMatchDetails and displaySets in the options object, fixing the issue where the custom attribute function 'sameAs' was missing required data. * feat(measurements): add CircleROI Tool (#3211) * [refactor] measurement service mapping files - rename to .ts files * [fix] RectangleROI - measurement service mapping _getReport() function - tool name fix * [feat] added CircleROI tool from updated cornerstone3D * [refactor] fix for typescript strong typing warnings * [feat] cornerstone-dicom-sr extension - added CircleROI tool * [feat] added toolbar button for CircleROI tool * [doc] doc updates for CircleROI tool * [update] library - dcmjs to 0.29.5 * [fix] roundNumber() function when given strings * [fix] refactor after upgrading conerstonejs library 0.49.0 * yarn.lock file change * fix: Service consistency typing (#3309) * Use more consistent type/structure for services * Fix restoring SR so the tests described work * One more typed service * Added types for the Cornerstone library * Couple more type fixes * feat: Add a new hanging protocol @ohif/mn (#3305) * feat: Add a new hanging protocol @ohif/mn * Add @ohif/seg example * PR comments and a couple more fixes to make things cleaner * PR comment * Added an example to cause the PR checks to rerun

…ical-4d-base-merge (#3368) * fix(volumeLoad): should not have missing slices when loading (#3287) * fix(volumeLoad): should not have missing slices when loading * add review comments * feat(DoubleClick): double click a viewport to one up and back (#3285) * feat(DoubleClick): double click a viewport to one up and back Added a toggleOneUp command that puts the active viewport into a 1x1 grid layout and it toggles out of 'one-up' by restoring its saved 'toggleOneUpViewportGridStore' from the StateSyncService. Added double click customization for the Cornerstone extension with the default double click handling being the toggleOneUp command. Added a cypress test for the double click functionality. * PR feedback: - tracked viewport measurements no longer show as dashed when toggling one up - disallowed double clicking near a measurement - updated cornerstone3D dependencies to fix double click of TMTV and volume viewport 3D - created ViewportGridService.getLayoutOptionsFromState * Updated the ViewportGridService docs. * Switched to using 'cornerstoneViewportClickCommands' and consistency with the context menu clicks. * feat(tmtv): add more stages to pt/ct (#3290) * feat(tmtv): add more stages to pt/ct * make error stage change to info * apply review comments * fix(viewports): The display of linked viewports during drag and drop has a race (#3286) * fix: The display of linked viewports during drag and drop has a race * PR review comments * fix: Segmentation display two up * Removing console logs * Fix the blank viewport can have stuff added to it * Fix the null name on HP module * Fix the navigate to initial image * Fix the nth interleave loader * Fix the unit tests * PR comments - docs mostly * fix: Exception thrown on change displayset after double click * feat(multiframe): enhanced support for multiframe dicom (#3164) * Changes in cswil version and multiframe * Minor changes * wip * Adding support for NM multiframe images * Applying PR suggestions * fixing package versions * Restoring default.js config file * Check if NM subtype is reconstructable * Restore default.js values * refactore code * feat: add flag for strict zspacing --------- Co-authored-by: Alireza <[email protected]> * feat(URL): add param for initial series and sop uids to display (#3265) * feat: Allow navigating to a specified series and sop instance This was a feature in OHIF v2, so adding it to v3, albeit with new parameters. feat: Allow comma separated as well as repeated args params * docs * Test fixes * feat: Navigate to SOP selected - PR fixes * Updated docs * PR fixes * fix(crosshairs): suppressed error for crosshair (#3237) * fix(SRTool): Ellipse Display for DICOMSR Tool (#3307) * feat(App): support async config function (#3313) * fix(URL): allow multi filter query (#3314) * fix(viewport): Initial blank image on SR/SEG initial display (#3304) * fix: Blank display area on initial DICOM SR load * Docs * Fix a NPE * fix(ROIThreshold): fix setWeight not updating properly for ROIThreshold panel (#3315) * fix(hp): Add displaySetMatchDetails and displaySets to options in (#3320) The sameAs function requires displaySetMatchDetails and displaySets to compare attributes between display sets, but these properties were not being passed into the options object. This commit adds the necessary lines of code to include displaySetMatchDetails and displaySets in the options object, fixing the issue where the custom attribute function 'sameAs' was missing required data. * feat(measurements): add CircleROI Tool (#3211) * [refactor] measurement service mapping files - rename to .ts files * [fix] RectangleROI - measurement service mapping _getReport() function - tool name fix * [feat] added CircleROI tool from updated cornerstone3D * [refactor] fix for typescript strong typing warnings * [feat] cornerstone-dicom-sr extension - added CircleROI tool * [feat] added toolbar button for CircleROI tool * [doc] doc updates for CircleROI tool * [update] library - dcmjs to 0.29.5 * [fix] roundNumber() function when given strings * [fix] refactor after upgrading conerstonejs library 0.49.0 * yarn.lock file change * fix: Service consistency typing (#3309) * Use more consistent type/structure for services * Fix restoring SR so the tests described work * One more typed service * Added types for the Cornerstone library * Couple more type fixes * feat: Add a new hanging protocol @ohif/mn (#3305) * feat: Add a new hanging protocol @ohif/mn * Add @ohif/seg example * PR comments and a couple more fixes to make things cleaner * PR comment * Added an example to cause the PR checks to rerun * chore(version): updated Orthanc from 1.5.7 to 1.11.0 (#3330) * fix(SR): When loading DICOM SR, only one measurement is shown with no way to show others (#3228) * fix: Make the cornerstone sR viewport show all measurements * PR fixes * PR fixes * Add a DICOM SR hanging protocol * Duplicate the hanging protocol for seg as well * PR requested change * PR requested changes * PR fixes plus merge update fixes * PR fixes and integration test fix * PR - documentation * feat(RT): add dicom RT support via volume viewports (#3310) * feat: initial RT support * make the segmentation service work with representation data * feat: make segmentation service work with representations * fix rtss vis * fix: rt hydration * fix the rendering of rt names * fix imports * refactor: Modify status and click handling for hydration of RTStructures Modify status and click handling for hydration of RTStructures by renaming `onPillClick` to `onStatusClick` in `OHIFCornerstoneRTViewport.tsx` and `_getStatusComponent.tsx` files. Also, update initial segmentation configurations in `PanelSegmentation.tsx` and simplify configuration changes and values for segmentation service in `SegmentationService.ts`. Finally, remove console debug in `CornerstoneViewportService.ts`. * wip for highlighting contours * refactor rt displayset code * review code update * update cornerstone dependencies * refactor: Update license year, version number, and minor code cleanup This commit updates the license year in several files, updates the version number in package.json, and contains minor code cleanup in two files. * add bulkdataURI retrieve for RT * fix package version * apply review comments * apply review comments * apply review comments * feat(panels): refactor and streamline segmentation configuration and inputs Rewrote state hooks and streamlined the configuration input for `PanelSegmentation` to be more verbose and reusable. Included several new input types, including the `InputRange` component which now shows a fixed floating value based on the step provided. The `SegmentationConfig` component now works with dynamic values controlled by `initialConfig`. These changes should improve function usability and make the code more maintainable going forward. * fix various bugs * fix contour delete by upgrade cs3d version * feat(viewport, inputNumber, segmentationConfig, orthanc): Implement minimum and maximum values for input number components, and useBulkDataURI for Orthanc configuration. Compare measurement view planes with absolute viewport view planes in Cornerstone viewport. * update yarn lock * feat(dicomImageLoader): replace wado image loader with the new library (#3339) * use new dicom image loader instead of cswil new dicom image loader fails update dicom image loader update yarn lock modify webpack * update webpack * fix webpack * update package versions * update package versions * fix(Browser history): fixed NPE when navigating a study via browser history and history navigation is now available via the navigateHistory command (#3337) * fix(Browser history): - fixed an NPE when navigating to a different study via the URL - exposed browser history navigation via a command * Added documentation for the navigateHistory command. Moved the history object from UI to viewer. * feat(storybook): Refactor Storybook to use Typescript (#3341) * feat(storybook): Refactor Storybook to use Typescript This commit updates the Storybook configuration to use Typescript, including renaming `main.js` to `main.ts`, updating `core.builder` and `framework.name` to use `@storybook/builder-webpack5` and `@storybook/react-webpack5`, respectively. The code also adds options to `addon-docs` to enable adding GFM support to the generated docs. Additionally, the commit modifies `staticDir`, removes outdated addons, and makes various devDependencies and PostCSS updates to align with Storybook 7.x.x. Finally, the commit removes some unused code and relocates `Button` and `AboutModal` story files. * apply review comments * update yarn lock * feat(microscopy): add dicom microscopy extension and mode (#3247) * skeleton for dicom-microscopy extension * skeleton for microscopy mode * [feat] ported @radicalimaging/microscopy-dicom to OHIF's default extension * [feat] ported microscopy mode from private repo * added new definitions to the package.json * webpack configuration update for microscopy extension * register new icons for microscopy tools * fixes to the microscopy extension and mode * trivial error fix - typescript type import error * link microscopy extension with default OHIF app plugin config * demo config fix * fix logs * remove unsed imports * [fix] loading of microscopy * [fix] webworker script loading, normalizing denaturalized dataset * found the latest version of dicom-microscopy-viewer that works with current OHIF extension : 0.35.2 * hide thumbnail pane by default, as we have issues with * comments * remove unused code * [feat] microscopy - annotation selection * [feat] microscopy - edit annotation label * wip * [bugfix] dicom-microscopy tool * [bugfix] dicom microscopy annotations * [fix] mixed-content blocking caused by BulkDataURI * [fix] microscopy measurements panel - center button * [fix] microscopy measurements panel - styling * [fix] microscopy - controls * fix local loading of microscopy * fix local loading of dicom microscopy * fix typo - indexof to indexOf * [fix] remove unused icons * remove commented out lines from webpack configuration * platform/viewer/public/config/default.js - revert accidental changes * [refactor] redirecting to microscopy mode on Local mode * attribution to DMV and SLIM viewer * [fix] code review feedbacks * fix thumbnails * [fix] microscopy - fix old publisher.publish() to PubSubService._broadcastEvent() * [fix] interactions, webpack config, roi selection * [feat] microscopy - remove select tool from UI * microscopy author * [fix] saving and loading SR * [bugfix] - missing publish() function in MicroscopyService, replace with _broadcastEvent * remove author section from readme * refactor SR saving feature * fix webpack config after merge * [bugfix] repeated import * fix e2e * webpack configuration * hide "Create report" button for microscopy * fix(ui): updated input fields to match new color scheme (#3323) * Updated input fields to match new color scheme Added small input text fields and fixed coloring * Added tailwind changes to viewers file * feat(cst): Add new command to get nearby annotation tools (#3327) * feat(cst): Add new command to get nearby annotation tools This commit adds a new command `getNearbyAnnotation` to the `commandsModule.ts` file that identifies nearby annotation tools excluding Crosshairs and ReferenceLines. It also removes an unused mapping to `Crosshairs` in `initMeasurementService.js` and changes the command run in `findNearbyToolData.ts` to `getNearbyAnnotation`. * perf(cornerstone): Improve performance by optimizing tools Removes unnecessary imports to improve app's performance. Changes how annotations are detected on the `commandsModule.js` file by fetching the `isAnnotation` property of the tool instance if available. Updates `CrosshairsTool` and `ReferenceLinesTool` to not be annotations anymore. Adjusts the `localhost` URL ports for the `dcm4chee-arc` service interface on `local_dcm4chee.js` to allow for more efficient server communication. * feat(cornerstone): Add CornerstoneServices type and ToolGroupService.getToolGroup to get tool group by ID or active viewport This commit introduces the new CornerstoneServices interface to the code and adds the ToolGroupService.getToolGroup method, which can retrieve a specific tool group by ID or from the active viewport. The older _getToolGroup method has also been removed from commandsModule. When no tool group ID is provided, this method now retrieves the tool group from the currently active viewport with the help of getActiveViewportEnabledElement. Additionally, the required reference to @ohif/core has been removed. * fix(viewportDialog): viewportDialoge not appearing in non-tracked viewports (#3071) * fix: viewportdialoge not appearing in non-tracked viewports * feat(viewports): Introduce useViewportDialog and remove deprecated API This commit introduces the `useViewportDialog` hook and replaces the deprecated `viewportDialogApi` with the new `viewportDialogState`. Additionally, the notifications in `OHIFCornerstoneRTViewport` and `OHIFCornerstoneViewport` have been removed. Finally, the `CinePlayer` component now accepts optional parameters. * fix tests * feat(DICOM Upload): Added new DICOM upload dialogue launched from the worklist page (#3326) * feat(DICOM Upload) OHIF #3297 - DicomWebDataSource.store.dicom now accepts an ArrayBuffer of data sets to store - DicomWebDataSource.store.dicom now also accepts optional callbacks to track upload and an AbortSignal object to cancel upload - Added DicomFileUploader class that performs and tracks the upload of a DICOM file - Added various UI pieces for the upload: DicomUpload, DicomUploadProgress, DicomUploadProgressItem - ProgressLoadingBar was extracted from LoadingIndicatorProgress so it can be reused - Modal dialogues can now optionally prevent an outside click from closing the Modal * Passing an XMLHttpRequest to the dataSource.store.dicom method instead of callbacks and AbortSignal. Cleanup of various UI pieces to minimize code and increase readability. * Made the DicomUpload component a customization module exported by the cornerstone extension. * Exposed a copy of the data source configuration via the IWebApiDataSource interface. Added dicomUploadEnabled to a data source's configuration. * Code clean up. * Upgraded the dicomweb-client version to one that provides the ability to pass a custom HTTP request. DICOM upload uses that custom HTTP request to track progress and cancel requests. * Distinguished between failed and cancelled uploads. * Allow no selection in the upload dialogue. Fixed the styling of various progress information so that everything aligns. * Switched from cornerstone wado image loader to cornerstone dicom image loader for DICOM upload. * Added special cancelled icon to differentiate from failed. * Added a bit of spacing between the upload progress bar and percentage. * Fixed minor issue with upload rejection. * Performance improvement for cancel all uploads: - use React memo for each upload item progress (row) - do not await each request of a cancel all * Fixed various padding/spacing for the DICOM upload drop zone component. Changed the border dashing for the DICOM upload drop zone to be a background image gradient. Added hover and active effects to the 'Cancel All Uploads' text. * fix(jump): Jump to measurement wasn't jumping when presentation already existed (#3318) * fix: Jump to measurement after presentation store * Fix initial load of DICOM SR * Fix measurement highlight issues * Clear measurements changed to array * Fix merge issues * PR comments * Removed unused formatting. Change to single event * Fix the unit test * PR review comments * refactor(viewport): simplify imports, align destructuring Simplify import statements by removing unused imports, organize existing imports, and align destructured imports consistently across the component. --------- * feat(proxy datasource): JSON server configuration launch (#2791) * feat: delegating dicom web proxy datasource * fix: configurably allow WADO image loader to issue preflight OPTIONS request * Added documentation and fixed up some error handling from PR review * review comments changes * another small code review fix * chore(lerna): upgrades lerna to 6.x (#3215) * feat(core): upgrades lerna to 6.x.x * feat(core): upgrades lerna to 6.5.1 * feat(core): updates lerna to 6.x.x * feat(core): updates default config * fix(DICOM Upload): Decreased the minimum height for the upload dialog contents to better fit some laptops. (#3347) * fix(HangingProtocolService): Add callback property & update description (#3349) The description for the numberOfDisplaySetsWithImages property was corrected. The update also involves adding a callback property that returns several objects (matchedViewports, viewportMatchDetails, displaySetMatchDetails) and returning them from the method 'findViewportProtocols'. The refactored code also has better readability. * chore(lerna): add lerna caching to build and tests (#3350) * chore(bump cs3d): bump package and add caching to webpack (#3353) * chore(bump cs3d): bump package and add caching to webpack * remove test for now * fix: conflicts ---------

…has a race (OHIF#3286) * fix: The display of linked viewports during drag and drop has a race * PR review comments * fix: Segmentation display two up * Removing console logs * Fix the blank viewport can have stuff added to it * Fix the null name on HP module * Fix the navigate to initial image * Fix the nth interleave loader * Fix the unit tests * PR comments - docs mostly * fix: Exception thrown on change displayset after double click

wayfarer3130 requested a review from sedghi March 28, 2023 18:22

wayfarer3130 requested a review from jbocce March 28, 2023 18:23

wayfarer3130 commented Mar 28, 2023

View reviewed changes

extensions/cornerstone-dicom-seg/src/utils/_hydrateSEG.ts Show resolved Hide resolved

wayfarer3130 commented Mar 28, 2023

View reviewed changes

extensions/cornerstone/src/services/ViewportService/CornerstoneViewportService.ts Show resolved Hide resolved

wayfarer3130 commented Mar 28, 2023

View reviewed changes

platform/ui/src/contextProviders/ViewportGridProvider.tsx Show resolved Hide resolved