fix(createImageRenderingActor): only clear color ranges if dirty

Kitware · Mar 27, 2023 · a11ef22 · a11ef22
1 parent 0fdc16a
commit a11ef22
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 10 deletions.
diff --git a/examples/compare-image.html b/examples/compare-image.html
@@ -22,9 +22,9 @@
 
         const fixedImage = new URL(
           // 'test-data/HeadMRVolume.nrrd',
-          // 'test-data/HeadMRVolumeLabels.nrrd',
+          'test-data/HeadMRVolumeLabels.nrrd',
           // 'test-data/HeadMRVolumeLabelsSmaller.nrrd',
-          'test-data/HeadMRVolume2Components.nrrd',
+          // 'test-data/HeadMRVolume2Components.nrrd',
           window.location.origin
         )
 

diff --git a/src/Rendering/Images/createImageRenderingActor.js b/src/Rendering/Images/createImageRenderingActor.js
@@ -37,8 +37,10 @@ const dirtyColorRanges = (c, { data: { name } }) => {
 
 const cleanColorRanges = (c, { data: { name } }) => {
   const actorContext = c.images.actorContext.get(name)
-  actorContext.colorRanges = new Map()
-  actorContext.dirtyColorRanges = false
+  if (actorContext.dirtyColorRanges) {
+    actorContext.colorRanges = new Map()
+    actorContext.dirtyColorRanges = false
+  }
 }
 
 const assignUpdateRenderedName = assign({

diff --git a/src/Rendering/VTKJS/Images/applyRenderedImage.js b/src/Rendering/VTKJS/Images/applyRenderedImage.js
@@ -154,12 +154,6 @@ function applyRenderedImage(context, { data: { name } }) {
       context.images.colorTransferFunctions.get(component) ??
       vtkColorTransferFunction.newInstance()
 
-    if (actorContext.colorRanges.has(component)) {
-      const range = actorContext.colorRanges.get(component)
-      colorTransferFunction.setMappingRange(range[0], range[1])
-      colorTransferFunction.updateRange()
-    }
-
     context.images.colorTransferFunctions.set(component, colorTransferFunction)
 
     if (!context.images.piecewiseFunctions.has(component)) {