Repository navigation
W11 WP 11.3: a horizon column is Measured only where a step was observed (#1058, #1117) - #1330
Merged
Merged
Conversation
…s observed (#1058, #1117) The tracer published a column as Measured where it had seen no sky-to-ground step: a wall skipped over as rows matching neither sky nor surface read as the ground below it, and a roof or crown with sky under it read as open. A column that would be Measured is now UnknownContrast when the boundary sits more than 4 degrees under the row where the walk first noticed a departure, or when an unexplained dark run of 6 degrees or more stands above the boundary. The traced altitude is kept; the bin publishes as Unknown (90), which errs closed. Chart yard: false-open Measured bins 36 to 18, Measured share 0.924 to 0.885. Treeline: 5 to 2; bins 273 to 275 are no longer published. Stored traces are not re-extracted and no type changes meaning. Mutant: every bin published as Measured; the crown, profile and wall cases fail. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K7CHb44VFtZLGUDLF4us2p
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.
Plan 2, wave 11, WP 11.3. A partial fix for #1058: it halves the false-open bins and says what remains.
The horizon tracer published a column as Measured where it had seen no sky-to-ground step. A false-open bin tells the planner a target is visible through a wall or a tree.
The rule. A column that would be Measured becomes
UnknownContrastwhen either:The traced altitude is kept for the review; the bin publishes as Unknown (90). The rule errs closed.
What remains (18 on the chart yard). Dark floating runs of 2 to 5 degrees and a wall the walk never noticed, under the 6 degree floor (chart discs and wires live there, so lowering it needs a way to tell a disc from a branch); and three bins from the scanner's 1.8 degree registration offset. Treeline's
measured_is_honeststill fails (p99 3.43 against 2.0) on edge placement; its ruling stays with updated numbers.Cost. A dark disc 6 to 12 degrees tall in open sky now makes its bin Unknown. Three pinned assertions that asserted the false open are changed.
Stored traces are not re-extracted, and no enum, type or field changes meaning.
CONTRACT.mdstates the rule.Mutant (every column passes the step test): the crown, profile and wall cases fail.
Part of #1058. Part of #1117.
🤖 Generated with Claude Code
https://claude.ai/code/session_01K7CHb44VFtZLGUDLF4us2p