Repository navigation
Calibration CI: groups that fit, and a timeout for each - #987
Merged
Merged
Conversation
The release run's calibration jobs ran for hours: - conformance-sweeps: the ExpoWeibull's mrl_cb likelihood-ratio sweep. Its mean residual life is an integral at every point the search tries, in a three-parameter valley; one two-sided bound at one time took 414 s on four cores, and the sweep asks for dozens. It is left out, as the offset, LFP and ZI variants' already were for the same reason; mrl_cb[lr] is still swept on 36 cases, the Weibull's taking the same path in closed form. The rest of the sweep: 242 tests, 10.4 minutes on four workers locally. - extra-parameters: its four studies took 52 minutes in one group (offset 1284 s, on the edge 903, lfp 651, zi 321). They run in three groups now, offset, on the edge, and lfp with zi, none of them cut. Every calibration group gets timeout-minutes: 45, so a hang fails in three quarters of an hour rather than GitHub's six. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019Km2eWBiPJ5JNkme3vRg4B
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
1 participant
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 release run's calibration jobs (#984) ran for hours.
mrl_cb[lr]sweep. Its mean residual life is an integral at every point the search tries, in a three-parameter valley. One two-sided bound at one time took 414 s locally on four cores, and the sweep asks for dozens of them.mrl_cb[lr]already were for the same reason.mrl_cb[lr]is still swept on 36 cases, and the Weibull's takes the same path in closed form.timeout-minutes: 45, so a hang fails fast instead of running for GitHub's six hours.Separately: a 414 s ExpoWeibull
mrl_cb(method="lr")is slow for users too. That is noted as follow-up, not changed here.Local: fast conformance passes; the sweep passes (242) in 10.4 min.
🤖 Generated with Claude Code
https://claude.ai/code/session_019Km2eWBiPJ5JNkme3vRg4B