Skip to content

Remove duplicate citation - #132

Merged
Hendrik-code merged 2 commits into
mainfrom
nm/remove_double_citation
Oct 12, 2026
Merged

Hendrik-code merged 2 commits into
mainfrom
nm/remove_double_citation

Conversation

@NathanMolinier

Copy link
Copy Markdown
Collaborator

What does this change?

Remove duplicate citation

Hendrik-code added a commit that referenced this pull request Oct 12, 2026
Eight CPU augmentations are batchgeneratorsv2 classes registered as they come,
so their __init__ signatures are what smauglab/configs/all_augmentations.json
records and what `smauglab matrix --check` compares the checked-in file against.
0.3.6 changed SpatialTransform: bg_style_seg_sampling flipped from True to
False and a keyword-only seg_tiebreak was added. That alone made the shipped
template stale, so all five test jobs failed on PR #132, a README-only change.

Capping instead of regenerating keeps the template matching the version the
paper configs were produced against. Training behaviour is unaffected either
way -- nnUNetTrainerTest passes bg_style_seg_sampling=False explicitly.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@Hendrik-code
Hendrik-code merged commit a53eaef into main Oct 12, 2026
8 checks passed
@Hendrik-code
Hendrik-code deleted the nm/remove_double_citation branch October 12, 2026 12:10
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