Conversation
…tems When using ensemble presets with custom_output_names, intermediate per-model separations received custom names that replaced the _(StemType)_ filename markers. This broke stem type classification (regex extraction), causing all stems to be labeled "Unknown"/"Other" and custom_output_names to not match. Fix: pass None to _separate_file for intermediate ensemble files (matching how _process_with_chunking already works), apply custom_output_names only to the final ensembled output. Bumps version to 0.43.1. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Documents how the auto-scaling GPU runner system works, including architecture, troubleshooting steps, and the critical requirement to update branch protection rules when renaming integration test jobs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Summary
custom_output_names, intermediate per-model separations received the custom names, which replaced the_(StemType)_filename markers needed for stem classificationcustom_output_namesdidn't match the final outputFix
One-line change in
_separate_ensemble(): passNoneinstead ofcustom_output_namesto per-model_separate_file()calls. Custom names are still applied to the final ensembled output (line 1348). This matches how_process_with_chunkingalready handles it.Test plan
tests/reproduce_ensemble_bug.py)tests/test_ensemble_fix_local.py)TestEnsembleCustomOutputNamesintests/unit/test_stem_naming.pytests/reproduce_ensemble_bug.pyagainst prod to verify fix@coderabbitai ignore
🤖 Generated with Claude Code