Repository navigation
Native route admits on main: own the three unowned fatal causes; record the class - #12626
Merged
Merged
Conversation
…frontier from measuring on main; record the class Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (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
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.
Work item adhoc-93d78779-4bb: why
//gunbc/instruments:v2-native-frontierrefuses its own admission on main.Diagnosis
The refusals in the brief do not reproduce on main.
positive_population_empty,required_native_pass_regressedand both unobserved controls were measured at eb5987e and cfc470c.Main 1457bd8, measured with
claim_executor --v2-native-routeon BuildBuddy using the #12550 recipe:v2.test.native_lane_smoke native_route_positive_population_smokepasses.v2.native_lane_fixture.controlis observed.unattributed_exclusions_presentandunattributed_file_refusals_present.The earliest unjustified boundary is the ownership table
v2.workflow.compile_door_cause_ownershipknown_frontier_causes. Three fatal reasons have producers but have never had a FatalGrain row:body_lowering_reason_unsupported_formbody_lowering_reason_function_value_body_unreadinfer_match_shape_invalidv2.compiler.04_inferinfer_match_bool_arm_row(older; newly reached)gunbc.witness_v2_native_routenative_route_exclusions_all_classifiedandnative_route_file_refusals_attributed. Only the native route runs that join, and the native route is off the merge path (gunbc.rung_drop v2_native_route_off_the_merge_path). So each of those PRs landed without the row it owed.v2.std.logicat context, was cascading through imports and emptying the positive population. The parent of MQ-1 PR-1: a caret symbol in value position refuses at the caret instead of lowering to its ^ token #12392 (19f039b) disproved it: there the smoke member reached eval and refused witheval_rejected_invalid_node. That was a separate eval defect, and it is fixed on main. The caret refusals remain real frontier debt; they are owned, and MQ-1 PR-2: a caret symbol lowers to a symbol literal (retire PR-1's refusal) #12420 retires them.gunbc.rung_drop native_lane_live_pair_expected_red: already retired (gunbc#12009). The pair is observed on main, andLivePairRequiredis the correct standing, so no change is needed.What this admission stands on: n = 1
NativeTestPassed, the smoke memberv2.test.native_lane_smoke native_route_positive_population_smoke, whose body istrue. The other 4,694 members are typed exclusions: refused at Context, Prepare or Eval.native_route_admissionsets no floor beyond non-empty. It has two positive-side clauses:native_route_positive_population_presentcheckscensus.agreement_pass > 0;native_route_required_passes_holdrequires every identity innative_route_required_pass_identitiesto pass. That roster holds one identity, the same smoke member.v2_native_route_off_the_merge_path, will stand on n = 1 until that exists.Change
CauseOwnershiprows, each with a lane and a flip trigger that names the capability.unsupported_formis a catch-all reason, so its trigger is that each producing site gets a reason naming its shape. A zero count alone is not the trigger.gunbc.recurring_failure_mode.refusal_producer_lands_without_its_ownership_row:Evidence (same recipe, pinned pushed shas, build and run in one dispatch, 24 GiB cgroup bound after the build)
unattributed_exclusions_present, unattributed_file_refusals_present[native-frontier-roster]proposaldriver-rows.jsonlare identical on base and head: 1NativeTestPassed, and the same refused counts per (stage, cause). The change moves attribution only; the population is unchanged.//gunbc/instruments:v2-native-frontieritself now answersFrontierUnmintedrather than NOT A MEASUREMENT. The roster is unminted by design (gunbc.native_frontier_debt), and minting belongs to the required lane.🤖 Generated with Claude Code