Skip to content

Vs pacs pr2mask labels - #89

Merged
skaliy merged 3 commits into
mainfrom
vs-pacs-pr2mask-labels
Oct 8, 2026
Merged

skaliy merged 3 commits into
mainfrom
vs-pacs-pr2mask-labels

Conversation

@skaliy

@skaliy skaliy commented Oct 8, 2026

Copy link
Copy Markdown
Member

No description provided.

skaliy added 3 commits October 8, 2026 09:07
The Research PACS filed our mask on its own study row and showed it
almost black: the mask kept the source StudyID and MR window. pr2mask
already writes the same mask in labels/ with the StudyID and window used
by fused and reports, so publish that series with our provenance tags.
Our own mask is now the intermediate model_mask.

- Remove the registered DICOM UID prefix; no published series used it.
- REDCap instance 1 holds the original mask and instances are filled in
  order: UNet 2, DynUNet 3, 4 reserved, nnU-Net 5. Instance 1 is
  rejected.
- The pr2mask UID seed is <version>_<model>_<tta|no-tta>. The report
  line drops the model, which the title already shows, and fused and
  vote map get the report title instead of the identity line.
- Set CONTAINER_VERSION so pr2mask records the image in SoftwareVersions.
@skaliy
skaliy force-pushed the vs-pacs-pr2mask-labels branch from 58f9aea to 3c2319c Compare October 8, 2026 07:08
@skaliy
skaliy merged commit b62d769 into main Oct 8, 2026
2 checks passed
@skaliy
skaliy deleted the vs-pacs-pr2mask-labels branch October 8, 2026 07:17
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.

1 participant