Repository navigation
Feat/vs pacs all regions - #97
Merged
Merged
Conversation
The integration team switched the PACS entry to fastmonai-vsseg:latest, so a new release needs no config change.
Only the largest region reached REDCap, so a second tumour region (for example participant 300) was lost. Each bundle now owns a block of 100 pr2mask repeat instances and region k goes to base + k, so DynUNet fills 201, 202, ... in pr2mask's "label k of N" order. An empty mask fills 201 with blank columns to record the run. Every instance also gets the new model_version field on the REDCap form, e.g. "dynunet 5-fold, fastmonai-vsseg:<stamp>". pr2mask has no such field, so the exporter writes it and adds it to the data dictionary. BREAKING CHANGE: DynUNet results move from instance 11 to 201+. The registry now holds block bases (multiples of 100); old results at 11 are removed on the REDCap side.
REDCap cuts longer values. pr2mask's mask text covers the whole scan (about 105,000 characters for a 512x448x176 MPRAGE), so a cut value is undecodable. Send it blank and log it; the full mask is in the PACS labels series.
Reverts the blanking of values over 65,535 bytes. It isn't known whether the receiving REDCap cuts them, and blanking would throw away a mask that REDCap might store whole. The previous release sent the full value and its import works.
0.13.1 resamples the predicted mask onto the scan's own grid. With 0.12.0, masks and vote maps could sit up to half a slice off on clinical grids. Built as fastmonai-vsseg:20261110T082650Z.
Cuts repeated setup steps and long prose from the research READMEs and AGENTS.md. AGENTS.md now also covers nbdev_clean, the generated root README, how to run the VS tests, release folders and writing style. The VS README keeps the note on merging folds from different machines as a bullet, and the PACS README's model_version example names the current image.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.