Say what a gather costs on the line after it - #35
Merged
Conversation
There was a problem hiding this comment.
Sorry @CNSeniorious000, you've used your own review budget of 250,000 diff characters for the last 7 days.
You can request another review in 4 days and 17 hours by commenting @sourcery-ai review. Upgrade to get a review now.
This was referenced Aug 31, 2026
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.
The
asyncio.gatherbullet says how to overlap calls and stops there. Combined with "the LAST expression is echoed back to you … reaching for it first keeps cells short", the cheapest reading is: fan out, then echo the list.Which is what
gpt-5.6-terradoes, in cell after cell:One line is short, and it pastes back five full answers.
The rule goes here rather than in the return-channel bullet because this is where the fan-out decision is made, and
resis the name the model actually binds.Measured — 4/4 tasks, 3 arms, all complete
Carried with #27, #30 and #36 as one arm.
gpt-5.6-terra, band 2K–20K. Dump rate = share of cells surfacing ≥ 90% of what they consumed; median ratio in brackets.private_lending_33dhsa_medical_bill_liquidity_guard_30dbroadcast_exam_posture_breathing_32dfood_safety_dispute_33dImportant
#27 + #30 alone win two tasks and lose two — a wash. This arm wins 4/4, including both tasks where that pair regressed, and every median ratio lands under 0.25.
broadcast_examis the one to check: baseline was already fine there at 9.1%, #27 + #30 took it to 38.2%, this arm takes it to 3.6% — belowglm-5.3's 4.8% on the same task.For reference on the same band:
macaron-v1-venti5.5% pooled andglm-5.34.8% / 1.8%. Both already reduce without any of this;gpt-5.6-terrawas the only one of the three that did not.Note
Attribution between this and #36 is not separated — they were measured together, and the arm does not carry #34.