@aics/vole-app 2.12.0 → 2.13.1

package/es/aics-image-viewer/components/App/ChannelUpdater.js

@@ -31,6 +31,8 @@ var ChannelUpdater = function ChannelUpdater(_ref) {
       if (image && version > 0) {
         return effect(image);
       }
+      // react-hooks will check that `deps` match `effect`'s dependencies, so we can safely exclude `effect` here
+      // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [].concat(_toConsumableArray(deps), [image, version]));
   };
 
@@ -40,30 +42,30 @@ var ChannelUpdater = function ChannelUpdater(_ref) {
       view3d.setVolumeChannelEnabled(image, index, volumeEnabled);
       view3d.updateLuts(image);
     }
-  }, [image, volumeEnabled]);
+  }, [image, volumeEnabled, index, view3d]);
   useEffect(function () {
     if (image) {
       view3d.setVolumeChannelOptions(image, index, {
         isosurfaceEnabled: isosurfaceEnabled
       });
     }
-  }, [image, isosurfaceEnabled]);
+  }, [image, isosurfaceEnabled, index, view3d]);
   useImageEffect(function (currentImage) {
     return view3d.setVolumeChannelOptions(currentImage, index, {
       isovalue: isovalue
     });
-  }, [isovalue]);
+  }, [isovalue, index, view3d]);
   useImageEffect(function (currentImage) {
     return view3d.setVolumeChannelOptions(currentImage, index, {
       isosurfaceOpacity: opacity
     });
-  }, [opacity]);
+  }, [opacity, index, view3d]);
   useImageEffect(function (currentImage) {
     view3d.setVolumeChannelOptions(currentImage, index, {
       color: color
     });
     view3d.updateLuts(currentImage);
-  }, [color]);
+  }, [color, index, view3d]);
   var controlPoints = channelState.controlPoints,
     ramp = channelState.ramp,
     useControlPoints = channelState.useControlPoints;
@@ -75,22 +77,19 @@ var ChannelUpdater = function ChannelUpdater(_ref) {
     var gradient = controlPointsToLut(controlPointsToUse);
     currentImage.setLut(index, gradient);
     view3d.updateLuts(currentImage);
-  }, [controlPoints, ramp, useControlPoints]);
+  }, [controlPoints, ramp, useControlPoints, index, view3d]);
   useImageEffect(function (currentImage) {
     if (colorizeEnabled) {
       // TODO get the labelColors from the tf editor component
       var lut = new Lut().createLabelColors(currentImage.getHistogram(index));
       currentImage.setColorPalette(index, lut.lut);
-      currentImage.setColorPaletteAlpha(index, colorizeAlpha);
-    } else {
-      currentImage.setColorPaletteAlpha(index, 0);
+      // following effect will also run and call `updateLuts`
     }
-    view3d.updateLuts(currentImage);
-  }, [colorizeEnabled]);
+  }, [colorizeEnabled, index, view3d]);
   useImageEffect(function (currentImage) {
     currentImage.setColorPaletteAlpha(index, colorizeEnabled ? colorizeAlpha : 0);
     view3d.updateLuts(currentImage);
-  }, [colorizeAlpha]);
+  }, [colorizeEnabled, colorizeAlpha, index, view3d]);
   return null;
 };
 export default ChannelUpdater;