POC: bring viv-dicomweb-test work into SLIM and add basic bulk annotations support

[igoroctaviano]

Summary

POC port of viv-dicomweb-test into SLIM: Viv + Deck.gl slide viewing on dedicated /viv/... routes plus bulk simple annotations (polygons) wired through the same DICOMweb + DMV decode path as OpenLayers.

• DMV: feat: export bulk annotation and graphic-fetch APIs for Viv / non-OpenLayers consumers dicom-microscopy-viewer#237

What changed (vs master)

Verified against feat/viv-loader ↔ master in the Slim repo (18 files, ~4.3k insertions).

• Routing: New React Router paths that mirror existing case URLs but prefix /viv: /viv/studies/:studyInstanceUID/* and the GCP Healthcare store variant. ParametrizedCaseViewer passes isVivRoute and optional vivSettings from config when the pathname starts with /viv/.

• src/viv/ module: dicomLoader.ts (DICOMweb tiling + DMV integration), VivSlideViewport.tsx, loadBulkAnnotationLayers.ts, vivDisplayDefaults.ts, README.md. Includes request abort/cancel handling and viewport work (incl. 8-bit SM tile positioning).

• UI integration: CaseViewer.tsx switches the slide viewport to VivSlideViewport on Viv routes, adds VivSettingsDrawer and an annotation rail wired to AnnotationGroupList, and adjusts study URLs so navigation stays under /viv/….

• Dependencies: @deck.gl/*, @vivjs/layers, @vivjs/loaders, @luma.gl/*; dicomweb-client raised to ^0.11.2 (plus types/dicomweb-client). Overrides pin Deck versions; bun.lock updated.

• Build: craco.config.js — Babel transpilation for node_modules under @deck.gl, @luma.gl, @math.gl, @probe.gl, @hms-dbmi/viv.

• DICOM web client hook: DicomWebManager.applyToPrimaryDicomwebClient so Viv tiles can attach to the primary store’s requestHooks (same XHR pipeline as VolumeImageViewer).

• Config types: AppConfig.d.ts — optional vivSettings (selections, channelsVisible, contrastLimits, colors, initialViewState). There is no useViv flag; Viv is strictly /viv/... URLs.

• ICC profiles: iccProfilesPreference.ts and updates in SlideViewer.tsx so ICC enable/disable persists (fixes ICC behavior with the Viv path).

• Types: types/dicom-microscopy-viewer/index.d.ts extended for DMV APIs the Viv loader uses.

• App.tsx: registers the Viv routes.

Branch & local setup

• SLIM branch: feat/viv-loader.
• POC still expects a linked dicom-microscopy-viewer checkout with Viv-facing API changes until a matching release is published. From DMV (correct branch): bun link. From the Slim checkout: bun link dicom-microscopy-viewer, then bun install if needed. bun unlink dicom-microscopy-viewer in Slim restores registry installs. Details: src/viv/README.md.

How to test

1. Add this server URL: https://proxy.imaging.datacommons.cancer.gov/current/viewer-only-no-downloads-see-tinyurl-dot-com-slash-3j3d9jyp/dicomWeb

2. Open the Viv-prefixed study URL (same study/series as before, /viv after the origin):

   http://localhost:3000/viv/studies/2.25.106918873973188798943205935727506273925/series/1.3.6.1.4.1.5962.99.1.1088146757.1503397867.1637470719301.2.0

3. (Optional) In public/config/*.js, set vivSettings for channel/contrast/colors/initialViewState on /viv/....

4. Toggle annotation group visibility. Expect a long first pass while every polygon is built (~23 s in one run); after that, pan/zoom stays responsive and memory behaved better than earlier OpenLayers attempts for this POC.

Possible TODO's

• Faster annotation loads: stream or commit geometry incrementally as it’s decoded; push heavy parsing/decoding to workers without breaking coordinate math.
• Clear busy state: loading / progress (and cancel/retry) for long annotation jobs.
• Lazy / viewport-aware loading: bulk annotations for matching series still load all at once; explore per-view / LOD loading on huge cases.
• Tighter layering in src/viv/: split network/DICOMweb, decode (DMV), and Deck.gl layer build so dicomLoader / viewport files do not become one mega-module; document slide space → Deck props for the next contributor.
• One source of truth with OpenLayers path: align behavior and coordinates with SlideViewer / VolumeImageViewer; avoid duplicated fetch/decode when DMV already exposes primitives.
• Dependency hygiene: consume a published DMV release with Viv-facing exports instead of bun link, and note which DMV version this tracks.
• Annotations beyond bulk simple: no TID1500 SR, SEG, optical-path sidebar, or presentation states on /viv yet — prioritize or gate behind flags (see README limitations).
• Docs / example config: add a vivSettings snippet to public/config example docs (paths are /viv/..., not a boolean config switch).
• README vs code: align src/viv/README.md limitations with dicomLoader (8-/16-bit handling and mixed-bit-depth restriction per series).
• Automated checks: smoke tests (build + minimal Viv route) and, if feasible, a perf budget for toggling annotation groups on a fixed IDC fixture.
• Failure modes: clearer errors when WebGL/Deck fails, DICOMweb errors, or malformed annotation corpora—without wedging the case view.

[deepsource-io]

DeepSource Code Review

We reviewed changes in ee5464e...d136f1d on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade

Security   Reliability   Complexity   Hygiene

Code Review Summary

Analyzer	Status	Updated (UTC)	Details
JavaScript		May 7, 2026 3:44p.m.	Review ↗

---

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

[igoroctaviano]

@fedorov, now you can test this using /viv
e.g.
Compare http://localhost:3000/viv/studies/2.25.302737996345872783571112300080988167697/series/1.3.6.1.4.1.5962.99.1.1354739488.2060999382.1637737312032.2.0
with http://localhost:3000/studies/2.25.302737996345872783571112300080988167697/series/1.3.6.1.4.1.5962.99.1.1354739488.2060999382.1637737312032.2.0

[fedorov]

@igoroctaviano it would be helpful to indicate somewhere (in the "info" panel perhaps?) whether given instance is using viv or not.

I deployed the viewer with useViv: true at https://andrey-slim-test.web.app/studies/2.25.147491078891690248552488589436202370211/series/1.3.6.1.4.1.5962.99.1.1471805016.1464679391.1637854377560.2.0, and I have no clue if it worked and what backend is being used!

[igoroctaviano]

@igoroctaviano it would be helpful to indicate somewhere (in the "info" panel perhaps?) whether given instance is using viv or not.

I deployed the viewer with useViv: true at https://andrey-slim-test.web.app/studies/2.25.147491078891690248552488589436202370211/series/1.3.6.1.4.1.5962.99.1.1471805016.1464679391.1637854377560.2.0, and I have no clue if it worked and what backend is being used!

PR description outdated. Just use the viv route before /studies

[sonarqubecloud]

Quality Gate passed

Issues
46 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
2.4% Duplication on New Code

See analysis details on SonarQube Cloud