@cornerstonejs/tools 1.19.3 → 1.20.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cornerstonejs/tools",
-  "version": "1.19.3",
+  "version": "1.20.0",
   "description": "Cornerstone3D Tools",
   "main": "dist/umd/index.js",
   "types": "dist/esm/index.d.ts",
@@ -29,7 +29,7 @@
     "webpack:watch": "webpack --mode development --progress --watch  --config ./.webpack/webpack.dev.js"
   },
   "dependencies": {
-    "@cornerstonejs/core": "^1.19.3",
+    "@cornerstonejs/core": "^1.20.0",
     "lodash.clonedeep": "4.5.0",
     "lodash.get": "^4.4.2"
   },
@@ -52,5 +52,5 @@
     "type": "individual",
     "url": "https://ohif.org/donate"
   },
-  "gitHead": "e084999f115529c5d959fcc20f58a5985c733121"
+  "gitHead": "165495ec246e9618992b6e10e3577b4a28a91e34"
 }

package/src/tools/annotation/AngleTool.ts

@@ -658,7 +658,10 @@ class AngleTool extends AnnotationTool {
       const canvasCoordinates = points.map((p) => viewport.worldToCanvas(p));
 
       // WE HAVE TO CACHE STATS BEFORE FETCHING TEXT
-      if (!data.cachedStats[targetId]) {
+      if (
+        !data.cachedStats[targetId] ||
+        data.cachedStats[targetId].angle == null
+      ) {
         data.cachedStats[targetId] = {
           angle: null,
         };

package/src/tools/annotation/BidirectionalTool.ts

@@ -1073,7 +1073,7 @@ class BidirectionalTool extends AnnotationTool {
       // force to recalculate the stats from the points
       if (
         !data.cachedStats[targetId] ||
-        data.cachedStats[targetId].unit === undefined
+        data.cachedStats[targetId].unit == null
       ) {
         data.cachedStats[targetId] = {
           length: null,

package/src/tools/annotation/CircleROITool.ts

@@ -692,7 +692,7 @@ class CircleROITool extends AnnotationTool {
       // force to recalculate the stats from the points
       if (
         !data.cachedStats[targetId] ||
-        data.cachedStats[targetId].areaUnit === undefined
+        data.cachedStats[targetId].areaUnit == null
       ) {
         data.cachedStats[targetId] = {
           Modality: null,

package/src/tools/annotation/CobbAngleTool.ts

@@ -674,7 +674,10 @@ class CobbAngleTool extends AnnotationTool {
       const canvasCoordinates = points.map((p) => viewport.worldToCanvas(p));
 
       // WE HAVE TO CACHE STATS BEFORE FETCHING TEXT
-      if (!data.cachedStats[targetId]) {
+      if (
+        !data.cachedStats[targetId] ||
+        data.cachedStats[targetId].angle == null
+      ) {
         data.cachedStats[targetId] = {
           angle: null,
         };

package/src/tools/annotation/DragProbeTool.ts

@@ -166,7 +166,10 @@ class DragProbeTool extends ProbeTool {
       ),
     };
 
-    if (!data.cachedStats[targetId]) {
+    if (
+      !data.cachedStats[targetId] ||
+      data.cachedStats[targetId].value == null
+    ) {
       data.cachedStats[targetId] = {
         Modality: null,
         index: null,

package/src/tools/annotation/EllipticalROITool.ts

@@ -814,7 +814,7 @@ class EllipticalROITool extends AnnotationTool {
       // force to recalculate the stats from the points
       if (
         !data.cachedStats[targetId] ||
-        data.cachedStats[targetId].areaUnit === undefined
+        data.cachedStats[targetId].areaUnit == null
       ) {
         data.cachedStats[targetId] = {
           Modality: null,

package/src/tools/annotation/LengthTool.ts

@@ -662,7 +662,7 @@ class LengthTool extends AnnotationTool {
       // force to recalculate the stats from the points
       if (
         !data.cachedStats[targetId] ||
-        data.cachedStats[targetId].unit === undefined
+        data.cachedStats[targetId].unit == null
       ) {
         data.cachedStats[targetId] = {
           length: null,

package/src/tools/annotation/PlanarFreehandROITool.ts

@@ -691,7 +691,7 @@ class PlanarFreehandROITool extends AnnotationTool {
         const { data } = annotation;
         if (
           !data.cachedStats[targetId] ||
-          data.cachedStats[targetId].areaUnit === undefined
+          data.cachedStats[targetId].areaUnit == null
         ) {
           data.cachedStats[targetId] = {
             Modality: null,

package/src/tools/annotation/ProbeTool.ts

@@ -452,7 +452,10 @@ class ProbeTool extends AnnotationTool {
         ),
       };
 
-      if (!data.cachedStats[targetId]) {
+      if (
+        !data.cachedStats[targetId] ||
+        data.cachedStats[targetId].value == null
+      ) {
         data.cachedStats[targetId] = {
           Modality: null,
           index: null,

package/src/tools/annotation/RectangleROITool.ts

@@ -673,7 +673,7 @@ class RectangleROITool extends AnnotationTool {
       // force to recalculate the stats from the points
       if (
         !data.cachedStats[targetId] ||
-        data.cachedStats[targetId].areaUnit === undefined
+        data.cachedStats[targetId].areaUnit == null
       ) {
         data.cachedStats[targetId] = {
           Modality: null,

package/src/tools/displayTools/Contour/contourDisplay.ts

@@ -2,6 +2,7 @@ import {
   getEnabledElementByIds,
   Types,
   utilities as csUtils,
+  StackViewport,
 } from '@cornerstonejs/core';
 
 import Representations from '../../../enums/SegmentationRepresentations';
@@ -128,6 +129,13 @@ async function render(
   const contourData = segmentation.representationData[Representations.Contour];
   const { geometryIds } = contourData;
 
+
+  // We don't have a good way to handle stack viewports for contours at the moment.
+  // Plus, if we add a segmentation to one viewport, it gets added to all the viewports in the toolGroup too.
+  if (viewport instanceof StackViewport) {
+    return;
+  }
+
   if (!geometryIds?.length) {
     console.warn(
       `No contours found for segmentationId ${segmentationId}. Skipping render.`