Aging improvements#1169
Merged
gmarull merged 3 commits intocoredevices:mainfrom Apr 27, 2026
Merged
Conversation
jplexer
approved these changes
Apr 27, 2026
Restructure the aging test to better stress the system and validate
battery health:
1. Charge while cycling components for 4 hours, charge must complete
within 90 min. Always display the live charge status.
2. Wait for operator to unplug, fully charged.
3. Idle for 10 hours; fail if battery drops more than 6%.
4. Discharge to a safe shipping level (65%) using the backlight.
The previous drop-during-cycling check is replaced by a charge-complete
check (must reach 100% within 90 min and stay there). Charging is no
longer disabled at 100% so the battery remains topped off until the
operator unplugs at the end of the combined phase.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Wire the aging test up to the manufacturing test result registry and expose its outcome in the finished tests QR. The aging test is launched from the top-level mfg menu instead of from the per-mode test menu, so it isn't registered in the menu's availability table. The QR rendering bypasses that check for aging and suppresses the entry entirely when the test wasn't run, so the semi-finished QR (where aging never runs) doesn't pick up a misleading AGE:N. The aging app forces the result into the FINISHED bucket so it doesn't matter what mode was active last. The reported value is the idle-phase battery drop in percent (0 if the idle phase was never reached), giving a useful battery-health datapoint alongside the pass/fail flag. QR field is "AGE:<P|F>,<drop>". Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Add the current battery percentage to the QR header, immediately after the firmware version, so the line can capture battery state at the moment of test completion. New header format: "<serial>;<mac>;<version>;<battery%>". Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
gmarull
force-pushed
the
aging-improvements
branch
from
09d851c to
03d6c93
Compare
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
2 participants
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.
No description provided.