Skip to content

Conversation

@PaulHax
Copy link
Collaborator

@PaulHax PaulHax commented Nov 7, 2025

Implements automatic layering based on file name patterns. When loading files with matching names like base-image.nii and base-image.layer.nii, the layer file is automatically placed on top of the base image.

@PaulHax PaulHax force-pushed the auto-layer branch 3 times, most recently from 3c016b2 to ba84a2e Compare November 10, 2025 14:42
@PaulHax PaulHax marked this pull request as ready for review November 10, 2025 14:44
@PaulHax PaulHax requested a review from zachmullen November 10, 2025 14:49
Copy link
Contributor

@zachmullen zachmullen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there ever a need to have more than one additional layer, or is it always just 0 or 1 layer?

Implements automatic layering based on file name patterns.
When loading files with matching names like base-image.nii and base-image.layer.nii, the layer file is automatically placed on top of the base image.
@PaulHax
Copy link
Collaborator Author

PaulHax commented Nov 10, 2025

Is there ever a need to have more than one additional layer, or is it always just 0 or 1 layer?

There could be more than 1.

@zachmullen
Copy link
Contributor

There could be more than 1.

How does this design accommodate that?

Sort matching layers and segment groups alphabetically by filename to provide deterministic ordering. Users can control stacking order using numeric prefixes in filenames (e.g., file.layer.1.nii, file.layer.2.nii).

Updated documentation to clarify naming patterns and show that file formats don't need to match between base image and associated files.
@PaulHax PaulHax merged commit edf3b7c into config-race Nov 11, 2025
3 checks passed
@zachmullen zachmullen deleted the auto-layer branch November 11, 2025 16:18
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