Repository navigation
Hook: cost a root's kept build with the model's kept-start cell - #1304
Merged
Merged
Conversation
cmux's warm-distance model now keeps the p50 compile per tier and start kind (tiers_by_start, with counts; cmux ci/warm-model-selfcal). Every root the hook compares starts from its kept build, and a kept start compiles faster than the tier's mix of kept and seed starts (near: about 91 s against 105 s for the tier and 157 s from a seed). read_warm_model reads each tier's kept cell once it has 5 compiles, as cmux's predict() does; a sparse or bad cell, or a model without tiers_by_start, costs the tier as before. 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.
cmux's warm-distance model now keeps the p50 compile per tier and start kind (
tiers_by_start, with counts; manaflow-ai/cmux ci/warm-model-selfcal, PR linked below). Every root this hook compares starts from its kept build, and kept starts compile faster than the tier's mix of kept and seed starts. Over 243 owned admissions:read_warm_modeladdskept, the seconds per tier fromtiers_by_start[tier]["kept"]. A cell counts only once it has ≥5 compiles (WARM_MIN_START_ROWS, the same rule as cmux'spredict()) and a positive finite non-bool p50. A sparse or bad cell is skipped on its own, so it does not discard the whole model. A model withouttiers_by_start, and the default model, havekept: {}and cost the tier exactly as before.warm_root_costscosts a stamped root atkept.get(tier, tiers[tier]). The unknown and cold costs are unchanged.test_warm_root_costs_rank_roots_by_predicted_compile, mirroring cmux'stest_predict_uses_a_cell_from_five_rows_and_the_tier_below:p50of -1, a bool, missing, a stringn, a list or null all fall back to the tier, for that tier only;tiers_by_startfalls back to the tier.python3 scripts/test-glaeda-cmux-runner.py -k warmpasses.Backward compatible in both directions: this hook with an old model, and an old hook with the new model, which only adds a key.
🤖 Generated with Claude Code