Skip to content

Pass full aman into preprocess pipe#1528

Merged
mmccrackan merged 6 commits intomasterfrom
260120_full_aman
Apr 10, 2026
Merged

Pass full aman into preprocess pipe#1528
mmccrackan merged 6 commits intomasterfrom
260120_full_aman

Conversation

@mmccrackan
Copy link
Copy Markdown
Contributor

@mmccrackan mmccrackan commented Jan 20, 2026

Addresses #1323.

Replacement for #1346 since the preprocessing infrastructure changed and the code could be simplified. This allows passing in an unrestricted preprocessing axis manager into the pipeline run function so that the total number of detectors is preserved when running the multiple layers as opposed to the current implementation where the second layer only retains the number of detectors left after the first layer cuts.

Tested by running preproc_or_load_group with both layers and checking the dimensions of proc_aman prior to saving. Also tested with multilayer_load_and_preprocess and confirmed both calls to get_meta return the full dimensions.

@mmccrackan mmccrackan requested a review from msilvafe January 20, 2026 19:22
@mmccrackan mmccrackan merged commit 7c3b809 into master Apr 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants