fix(scroll): Scrolling failed to find the volume with segmentation

common/reviews/api/core.api.md

@@ -709,6 +709,7 @@ function getSpacingInNormalDirection(imageVolume: IImageVolume, viewPlaneNormal:
 function getTargetVolumeAndSpacingInNormalDir(viewport: IVolumeViewport, camera: ICamera, targetVolumeId?: string): {
     imageVolume: IImageVolume;
     spacingInNormalDirection: number;
+    uid: string;
 };
 
 // @public (undocumented)

common/reviews/api/tools.api.md

@@ -2943,6 +2943,7 @@ function jumpToSlice(element: HTMLDivElement, options?: JumpToSliceOptions): Pro
 type JumpToSliceOptions = {
     imageIndex: number;
     debounceLoading?: boolean;
+    volumeId?: string;
 };
 
 // @public (undocumented)

packages/core/src/utilities/getTargetVolumeAndSpacingInNormalDir.ts

@@ -25,12 +25,13 @@ export default function getTargetVolumeAndSpacingInNormalDir(
 ): {
   imageVolume: IImageVolume;
   spacingInNormalDirection: number;
+  uid: string;
 } {
   const { viewPlaneNormal } = camera;
   const volumeActors = viewport.getActors();
 
   if (!volumeActors || !volumeActors.length) {
-    return { spacingInNormalDirection: null, imageVolume: null };
+    return { spacingInNormalDirection: null, imageVolume: null, uid: null };
   }
 
   const imageVolumes = volumeActors
@@ -44,26 +45,29 @@ export default function getTargetVolumeAndSpacingInNormalDir(
 
   // If a volumeId is defined, set that volume as the target
   if (targetVolumeId) {
-    const imageVolume = imageVolumes.find(
+    const imageVolumeIndex = imageVolumes.findIndex(
       (iv) => iv.volumeId === targetVolumeId
     );
 
+    const imageVolume = imageVolumes[imageVolumeIndex];
+    const { uid } = volumeActors[imageVolumeIndex];
     const spacingInNormalDirection = getSpacingInNormalDirection(
       imageVolume,
       viewPlaneNormal
     );
 
-    return { imageVolume, spacingInNormalDirection };
+    return { imageVolume, spacingInNormalDirection, uid };
   }
 
   if (!imageVolumes.length) {
-    return { spacingInNormalDirection: null, imageVolume: null };
+    return { spacingInNormalDirection: null, imageVolume: null, uid: null };
   }
 
   // Fetch volume actor with finest resolution in direction of projection.
   const smallest = {
     spacingInNormalDirection: Infinity,
     imageVolume: null,
+    uid: null,
   };
 
   for (let i = 0; i < imageVolumes.length; i++) {
@@ -74,9 +78,12 @@ export default function getTargetVolumeAndSpacingInNormalDir(
       viewPlaneNormal
     );
 
-    if (spacingInNormalDirection < smallest.spacingInNormalDirection) {
+    // Allow for 1/1000 part larger requirement to prefer earlier volumes
+    // when the spacing is all essentially the same.
+    if (spacingInNormalDirection * 1.001 < smallest.spacingInNormalDirection) {
       smallest.spacingInNormalDirection = spacingInNormalDirection;
       smallest.imageVolume = imageVolume;
+      smallest.uid = volumeActors[i].uid;
     }
   }
 

packages/tools/src/types/JumpToSliceOptions.ts

@@ -1,6 +1,7 @@
 type JumpToSliceOptions = {
   imageIndex: number;
   debounceLoading?: boolean;
+  volumeId?: string;
 };
 
 export default JumpToSliceOptions;

packages/tools/src/utilities/cine/playClip.ts

@@ -103,7 +103,7 @@ function playClip(
       targetImageIdIndex: viewport.getTargetImageIdIndex(),
       imageIds: viewport.getImageIds(),
     };
-    
+
     let newImageIdIndex = stackData.targetImageIdIndex;
     const imageCount = stackData.imageIds.length;
 
@@ -182,6 +182,7 @@ function playClip(
  */
 function stopClip(element: HTMLDivElement): void {
   const enabledElement = getEnabledElement(element);
+  if (!enabledElement) return;
   const { viewport } = enabledElement;
 
   const cineToolData = getToolState(viewport.element);

packages/tools/src/utilities/scroll.ts

@@ -38,19 +38,20 @@ export function scrollVolume(
 ) {
   const camera = viewport.getCamera();
   const { focalPoint, viewPlaneNormal, position } = camera;
-  const { spacingInNormalDirection, imageVolume } =
+  const { spacingInNormalDirection, uid } =
     csUtils.getTargetVolumeAndSpacingInNormalDir(viewport, camera, volumeId);
 
-  if (!imageVolume) {
+  if (!uid) {
     throw new Error(
       `Could not find image volume with id ${volumeId} in the viewport`
     );
   }
 
-  const actorEntry = viewport.getActor(imageVolume.volumeId);
+  const actorEntry = viewport.getActor(uid);
 
   if (!actorEntry) {
-    console.warn('No actor found for with actorUID of', imageVolume.volumeId);
+    console.warn('No actor found for with actorUID of', uid);
+    return;
   }
 
   const volumeActor = actorEntry.actor as Types.VolumeActor;

packages/tools/src/utilities/viewport/jumpToSlice.ts

@@ -23,7 +23,7 @@ async function jumpToSlice(
   element: HTMLDivElement,
   options = {} as JumpToSliceOptions
 ): Promise<void> {
-  const { imageIndex, debounceLoading } = options;
+  const { imageIndex, debounceLoading, volumeId } = options;
   const enabledElement = getEnabledElement(element);
 
   if (!enabledElement) {
@@ -40,7 +40,7 @@ async function jumpToSlice(
   const imageIndexToJump = _getImageIndexToJump(numberOfSlices, imageIndex);
   const delta = imageIndexToJump - currentImageIndex;
 
-  scroll(viewport, { delta, debounceLoading });
+  scroll(viewport, { delta, debounceLoading, volumeId });
 }
 
 function _getImageSliceData(