Repository navigation
Fix main CI: let the CosyVoice3 flow capture test's stand-in take the stream argument - #1185
Merged
dghosef merged 1 commit intoNov 4, 2026
Conversation
… stream argument mstar-project#1073 added stream= to capture_into_graph; mstar-project#657's CPU stand-in predates it, so main's CPU Core fails test_cosyvoice3_flow_capture.py with a TypeError. Co-Authored-By: Claude Opus 5.5 <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
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.
Fixes main's CPU Core failure (run 41800682122).
What this does
test/modular/test_cosyvoice3_flow_capture.pyfails on main: all 10 CPU tests raiseTypeError: _runners.<locals>.<lambda>() got an unexpected keyword argument 'stream'.capture_into_graphnow takesstream=None, like the real function.How it works
#1073 added
stream=tocapture_into_graph, andcapture_with_static_outputspasses it through. The test came in with #657, which passed CI on 10-30, before #1073 landed, and was merged on top of it without a re-run. The stand-in's signature didn't have the new argument. This is a one-line test change; no runtime code changes.test_cosyvoice3_flow_capture.py, 20x🤖 Generated with Claude Code