[Bug] BidirectionalTool on volume viewport spams ANNOTATION_MODIFIED events

[Gabsha]

Describe the Bug

I am using Bidirectional annotations on volume viewport and noticed that the the ANNOTATION_MODIFIED event gets fired repeatedly on zooming and panning.

localhost_3000.-.Google.Chrome.2024-08-15.14-16-34.mp4

The event gets fired in _calculateCachedStats, which is called repeatedly in renderAnnotation because the targetId for a volume viewport is a URN string that changes every time the camera is updated (as defined in BaseVolumeViewport)

See:

cornerstone3D/packages/tools/src/tools/annotation/BidirectionalTool.ts

Line 1051 in 642bbba

!data.cachedStats[targetId] ||

(Willing to contribute a fix, but not sure the best course of action. Perhaps cached stats should ignore the focalPoint portion of the URN to verify existence.)

Steps to Reproduce

Steps are similar to #1294

1. Open niftiWithToolsexample: yarn run example niftiWithTools
2. Add an event listener in the setup function

eventTarget.addEventListener(csToolsEnums.Events.ANNOTATION_MODIFIED,(event)=>{
  console.count(`${event.type}: ${event.detail.annotation.metadata.toolName}`)
})

The current behavior

After placing a BidirectionTool annotation on a volume viewport, a ANNOTATION_MODIFIED event is fired when zooming and panning the viewport.

The expected behavior

The ANNOTATION_MODIFIED event is only fired when the annotation is actually modified (handles moved).

OS

Windows 10

Node version

20.16.0

Browser

Chrome 127.0.6533.101 (Official Build) (64-bit)