feat(contourSeg) polyline decimate :: code review

cornerstonejs · Feb 8, 2024 · aefdbb0 · aefdbb0
1 parent e00a59f
commit aefdbb0
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 7 deletions.
diff --git a/packages/tools/src/tools/annotation/LivewireContourTool.ts b/packages/tools/src/tools/annotation/LivewireContourTool.ts
@@ -113,7 +113,7 @@ class LivewireContourTool extends ContourSegmentationBaseTool {
          * for better performance and storage.
          */
         decimate: {
-          enabled: true,
+          enabled: false,
           /** A maximum given distance 'epsilon' to decide if a point should or
            * shouldn't be added the resulting polyline which will have a lower
            * number of points for higher `epsilon` values.

diff --git a/packages/tools/src/tools/annotation/PlanarFreehandROITool.ts b/packages/tools/src/tools/annotation/PlanarFreehandROITool.ts
@@ -236,7 +236,7 @@ class PlanarFreehandROITool extends ContourSegmentationBaseTool {
          * for better performance and storage.
          */
         decimate: {
-          enabled: true,
+          enabled: false,
           /** A maximum given distance 'epsilon' to decide if a point should or
            * shouldn't be added the resulting polyline which will have a lower
            * number of points for higher `epsilon` values.

diff --git a/packages/tools/src/tools/annotation/SplineROITool.ts b/packages/tools/src/tools/annotation/SplineROITool.ts
@@ -125,7 +125,7 @@ class SplineROITool extends ContourSegmentationBaseTool {
          * for better performance and storage.
          */
         decimate: {
-          enabled: true,
+          enabled: false,
           /** A maximum given distance 'epsilon' to decide if a point should or
            * shouldn't be added the resulting polyline which will have a lower
            * number of points for higher `epsilon` values.

diff --git a/packages/tools/src/tools/base/ContourBaseTool.ts b/packages/tools/src/tools/base/ContourBaseTool.ts
@@ -224,10 +224,6 @@ abstract class ContourBaseTool extends AnnotationTool {
     }
   ) {
     const decimateConfig = this.configuration?.decimate || {};
-    console.log('>>>>> decimate', {
-      enabled: !!decimateConfig.enabled,
-      epsilon: decimateConfig.epsilon,
-    });
 
     updateContourPolyline(annotation, polylineData, transforms, {
       decimate: {