Repository navigation
Save Auto-Calibrate results without asking - #104
Merged
Merged
Conversation
When the preflight found the radio unresponsive and restarted it, the calibrate window showed an orange note saying maximum attenuation had been saved to the configuration. It was shown twice on a successful run, right next to the Persist button, and read as a current fault rather than history (ClickUp 123zgec4dua). The run still records the restart in status["preflight"], and a run that cannot recover the radio still explains why in its own error message. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The calibrate window no longer offers a "Persist to config" button. When a run ends with tuning to keep (a confirmed result, or the no-track fallback that every Quick Calibrate ends with), it calls /calibrate/apply itself: the same user.yml write and the same apply queue the button used, with progress shown under the result. The button read as optional, when leaving the tuning unsaved only meant the next restart discarded the run (ClickUp 123zgec4dua). Cancelled runs have no tuning and save nothing, as before. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This was referenced Sep 29, 2026
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.
Summary
Two changes to the Auto-Calibrate window, for ClickUp 123zgec4dua ("Autoconfig Confusing"). A customer ran Auto-Calibrate, saw a successful result under two copies of an orange warning, pressed Persist to config, and was left unsure what they had just done.
status["preflight"]. A run that cannot recover the radio still explains why in its own error message./calibrate/applyitself. That covers a confirmed result, and the no-track fallback that every Quick Calibrate ends with. It's the same route, the same user.yml write and the same apply queue the button used, so the server side is unchanged. Progress shows on a line under the result, using the apply's own phase labels, and ends in "Saved and applied." A failure is shown in red. A cancelled run has no tuning, so it saves nothing, as before. The setup wizard already saved this way. Its only change is a comment.Behaviour to be aware of
Verification
ed5e9b7. Two new wiring tests cover the missing button and both run endings saving through/calibrate/apply. The test that located form-adoption code by the old button's click handler now uses the new function./calibrate/applyby itself, exactly once. It showed "Restarting the SDR service", then the 30 s settle countdown, then "Restarting radar services", then "Saved and applied", 46 s after the run ended.Test plan
🤖 Generated with Claude Code