Skip to content

Commit 05f4a6c

Browse files
docs(output): don't claim the docs define Limit as the end-of-window SoC
docs/predbat-plan-card.md describes the end of the slot, not of the window. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent f62569d commit 05f4a6c

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

‎apps/predbat/output.py‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1424,10 +1424,11 @@ def import_rate_color(rate):
14241424
target = self.charge_window_best[charge_window_n]["target"]
14251425

14261426
# Classify Chrg vs HoldChrg against the real instruction (charge_limit_best), but have a Chrg row
1427-
# display the window's predicted end-of-window SoC ("target", from plan.py), which is what the
1428-
# Limit column documents. Classifying against the prediction showed a full-rate charge that can't
1429-
# finish within the window as HoldChrg whenever the row's SoC sat above that prediction (issue #4445).
1430-
# A HoldChrg row shows the limit it is held at, which the prediction need not match.
1427+
# keep displaying the window's predicted SoC ("target", from plan.py) as before, rather than an
1428+
# instruction the window may never reach. Classifying against the prediction showed a full-rate
1429+
# charge that can't finish within the window as HoldChrg whenever the row's SoC sat above that
1430+
# prediction (issue #4445). A HoldChrg row shows the limit it is held at, which the prediction
1431+
# need not match.
14311432
instruction_percent = calc_percent_limit(limit, self.soc_max)
14321433
limit_percent = calc_percent_limit(target, self.soc_max)
14331434
if limit > 0.0:

0 commit comments

Comments
 (0)