Skip to content

Adding ROIs to Parsed Metadata#914

Draft
simonwarchol wants to merge 24 commits intohms-dbmi:mainfrom
simonwarchol:main
Draft

Adding ROIs to Parsed Metadata#914
simonwarchol wants to merge 24 commits intohms-dbmi:mainfrom
simonwarchol:main

Conversation

@simonwarchol
Copy link
Copy Markdown

@simonwarchol simonwarchol commented Sep 24, 2025

Addressing #913

Background

This PR adds ROI (Region of Interest) functionality to Viv by extending the OME-XML parsing logic.
With this change, ROI elements defined in OME-XML can be extracted and made available in Viv’s data model, allowing downstream tools to render and interact with ROI annotations.

Change List

  • Added ROI parsing support in the OME-XML parser.
  • Extended types/interfaces to include ROI data structures.
  • Updated documentation to note ROI support.

Checklist

  • Update JSdoc types if there is any API change.
  • Make sure Avivator works as expected with your change.

@dbmi-svc-checkmarx
Copy link
Copy Markdown

dbmi-svc-checkmarx commented Oct 7, 2025

Logo
Checkmarx One – Scan Summary & Detailsf9c5d695-b281-454e-8e1c-03ee8df30b18

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to interact with Checkmarx PR Assistant.
Examples:
@Checkmarx how are you able to help me?
@Checkmarx rescan this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants