feat(invertSync): add invert sync to voi sync (#677)

* feat(invertSync): add invert sync to voi sync * update api
cornerstonejs · Jul 4, 2023 · a1dcfbc · a1dcfbc
1 parent 5119d8f
commit a1dcfbc
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 10 deletions.
diff --git a/common/reviews/api/core.api.md b/common/reviews/api/core.api.md
@@ -2703,6 +2703,7 @@ type VoiModifiedEventDetail = {
     volumeId?: string;
     VOILUTFunction?: VOILUTFunctionType;
     invert?: boolean;
+    invertStateChanged?: boolean;
 };
 
 // @public (undocumented)

diff --git a/common/reviews/api/streaming-image-volume-loader.api.md b/common/reviews/api/streaming-image-volume-loader.api.md
@@ -1600,6 +1600,7 @@ type VoiModifiedEventDetail = {
     volumeId?: string;
     VOILUTFunction?: VOILUTFunctionType;
     invert?: boolean;
+    invertStateChanged?: boolean;
 };
 
 // @public (undocumented)

diff --git a/common/reviews/api/tools.api.md b/common/reviews/api/tools.api.md
@@ -5466,6 +5466,7 @@ type VoiModifiedEventDetail = {
     volumeId?: string;
     VOILUTFunction?: VOILUTFunctionType;
     invert?: boolean;
+    invertStateChanged?: boolean;
 };
 
 // @public (undocumented)

diff --git a/packages/core/src/RenderingEngine/BaseVolumeViewport.ts b/packages/core/src/RenderingEngine/BaseVolumeViewport.ts
@@ -333,6 +333,7 @@ abstract class BaseVolumeViewport extends Viewport implements IVolumeViewport {
         volumeId: volumeIdToUse,
         VOILUTFunction: this.VOILUTFunction,
         invert: this.inverted,
+        invertStateChanged: true,
       };
 
       triggerEvent(this.element, Events.VOI_MODIFIED, eventDetail);

diff --git a/packages/core/src/types/EventTypes.ts b/packages/core/src/types/EventTypes.ts
@@ -42,6 +42,8 @@ type VoiModifiedEventDetail = {
   VOILUTFunction?: VOILUTFunctionType;
   /** inverted */
   invert?: boolean;
+  /** Indicates if the 'invert' state has changed from the previous state */
+  invertStateChanged?: boolean;
 };
 
 /**

diff --git a/packages/tools/src/synchronizers/callbacks/voiSyncCallback.ts b/packages/tools/src/synchronizers/callbacks/voiSyncCallback.ts
@@ -21,7 +21,7 @@ export default function voiSyncCallback(
   voiModifiedEvent: Types.EventTypes.VoiModifiedEvent
 ): void {
   const eventDetail = voiModifiedEvent.detail;
-  const { volumeId, range } = eventDetail;
+  const { volumeId, range, invertStateChanged, invert } = eventDetail;
 
   const renderingEngine = getRenderingEngine(targetViewport.renderingEngineId);
   if (!renderingEngine) {
@@ -31,18 +31,20 @@ export default function voiSyncCallback(
   }
 
   const tViewport = renderingEngine.getViewport(targetViewport.viewportId);
+  const tProperties:
+    | Types.VolumeViewportProperties
+    | Types.StackViewportProperties = {
+    voiRange: range,
+  };
+
+  if (invertStateChanged) {
+    tProperties.invert = invert;
+  }
 
   if (tViewport instanceof VolumeViewport) {
-    tViewport.setProperties(
-      {
-        voiRange: range,
-      },
-      volumeId
-    );
+    tViewport.setProperties(tProperties, volumeId);
   } else if (tViewport instanceof StackViewport) {
-    tViewport.setProperties({
-      voiRange: range,
-    });
+    tViewport.setProperties(tProperties);
   } else {
     throw new Error('Viewport type not supported.');
   }