Skip to content

Fix far-ladder kink misclassification; allow disabling the far ladder - #84

Open
dweindl wants to merge 1 commit into
mainfrom
fix/far-ladder-kink-misclassification
Open

dweindl wants to merge 1 commit into
mainfrom
fix/far-ladder-kink-misclassification

Conversation

@dweindl

@dweindl dweindl commented Sep 22, 2026

Copy link
Copy Markdown
Member

Summary

  • A discontinuity whose crossing distance falls inside the far ladder's own range could destabilize its extrapolation, inflating far_extrapolation.error_estimate enough to widen cross_regime's own budget past the disagreement it should flag, and far_discontinuity's check used a noise scale calibrated for the main ladder rather than the far ladder's own — together, this could misreport status="noise_dominated" for directions that are actually near a real discontinuity.
  • Fixes this by probing a noise floor local to the far ladder's own scale, scanning every adjacent-rung pair (not just the one anchored at best_index), and folding far_discontinuity.suspected into status.
  • Adds n_rungs_far=0 as a way to disable the far ladder (and its noise probe) entirely, for callers who'd rather trade that robustness for fewer evaluations.

🤖 Generated with Claude Code

A discontinuity whose crossing distance falls inside the far ladder's
own range could destabilize its extrapolation and get reported as
noise_dominated instead of discontinuity_suspected: the inflated
far_extrapolation.error_estimate widened cross_regime's own budget
enough to mask the disagreement, and far_discontinuity's check used a
noise scale calibrated for the main ladder, not the far ladder's own.

Fixes this by probing a noise floor local to the far ladder's own
scale, scanning every adjacent-rung pair (not just the one anchored at
best_index), and folding far_discontinuity.suspected into status.

Also lets n_rungs_far=0 disable the far ladder (and its noise probe)
entirely, for callers who'd rather trade that robustness for fewer
evaluations.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant