Deploy 0.18.2 to master #200

Merged
admin merged 6 commits from deploy-0.18.2 into master 2026-10-17 01:59:55 +02:00
Owner

Deploy add-on 0.18.2 (release/1.0 merge c67d0cd, PR #199) to master.

Contents: SAFETY-118 (peak forecast sensor read in its own unit). See goodwe_controller/CHANGELOG.md ## 0.18.2.

  • Owner go for deploy: PENDING. Do not merge without it.
  • predeploy_check (--deployed-version 0.18.1) on fresh live options: to be run before merge.
  • Merge only after checks / unit (pull_request) is green on this head, pinning head_commit_id.
  • No option change on deploy. After the update: check the live peak forecast sensor has a power unit (W/kW).

🤖 Generated with Claude Code

https://claude.ai/code/session_01Bdqr7PiaHcBnzPituu2yTC

Deploy add-on 0.18.2 (release/1.0 merge c67d0cd, PR #199) to `master`. Contents: SAFETY-118 (peak forecast sensor read in its own unit). See `goodwe_controller/CHANGELOG.md` `## 0.18.2`. - Owner go for deploy: PENDING. Do not merge without it. - predeploy_check (`--deployed-version 0.18.1`) on fresh live options: to be run before merge. - Merge only after `checks / unit (pull_request)` is green on this head, pinning `head_commit_id`. - No option change on deploy. After the update: check the live peak forecast sensor has a power unit (W/kW). 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01Bdqr7PiaHcBnzPituu2yTC
admin added 6 commits 2026-10-16 22:32:17 +02:00
SAFETY-118: read the peak forecast in its own unit
checks / unit (pull_request) Successful in 40m20s
infra-pin / fresh (pull_request) Successful in 1s
f0b4e68e49
peak_forecast_entity is read through pv.watts() by its unit_of_measurement;
the abs(raw) < 50 size rule is gone. No unit or a non-power unit means the
forecast is not used, with one warning per cause. The validator no longer
claims a kW sensor is taken as watts for this entity.

Co-Authored-By: Claude <noreply@anthropic.com>
SAFETY-118: a missing peak forecast reading keeps the unit-fault state
infra-pin / fresh (pull_request) Successful in 1s
checks / unit (pull_request) Successful in 40m19s
7d52dc1c0f
A missing reading (unavailable, unknown, failed read, non-numeric) no longer
clears the fault: no false 'used again' INFO, no repeated WARNING, the status
row stays. Adds pv.has_reading(). DECISIONS wording fixed; tests added.

Co-Authored-By: Claude <noreply@anthropic.com>
Release 0.18.2
checks / unit (pull_request) Successful in 40m23s
infra-pin / fresh (pull_request) Successful in 2s
75f4359aac
Version bump to 0.18.2 and a CHANGELOG section for SAFETY-118 (#198).

Co-Authored-By: Claude <noreply@anthropic.com>
CHANGELOG 0.18.2: say how a no-unit forecast shows; who needs a look
checks / unit (pull_request) Successful in 40m19s
infra-pin / fresh (pull_request) Successful in 1s
ea77c63552
Review of PR #199. "Warns once" left out the Home row and the Settings
note, which stay while the fault lasts. "No option change" now points to
the one case that needs a look.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Merge pull request 'Release 0.18.2' (#199) from release-0.18.2 into release/1.0
infra-pin / fresh (push) Successful in 1s
checks / unit (pull_request) Successful in 40m25s
infra-pin / fresh (pull_request) Successful in 1s
c67d0cdb67
admin merged commit ad374ae208 into master 2026-10-17 01:59:55 +02:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: admin/goodwe-addon#200