Skip to content

Add FAQ entries for power sensing, battery coordination, and troubleshooting#396

Merged
tomquist merged 3 commits into
developfrom
claude/magical-ritchie-R4jlc
May 27, 2026
Merged

Add FAQ entries for power sensing, battery coordination, and troubleshooting#396
tomquist merged 3 commits into
developfrom
claude/magical-ritchie-R4jlc

Conversation

@tomquist
Copy link
Copy Markdown
Owner

@tomquist tomquist commented May 27, 2026

Summary

Expanded the README FAQ section with practical guidance for common configuration and troubleshooting scenarios, covering power meter setup, multi-battery coordination, oscillation issues, and Marstek app integration.

Changes

  • Power sensing guidance: Added FAQ clarifying single signed sensor vs. separate import/export sensors for Home Assistant App, explaining sync drift risks
  • Multi-battery coordination: New entry recommending CT002/CT003 emulation over Shelly for multi-battery setups to prevent independent battery conflicts
  • Oscillation troubleshooting: Detailed guidance on diagnosing and tuning oscillating output power, covering THROTTLE_INTERVAL, DEDUPE_TIME_WINDOW, DEADBAND, SMOOTH_TARGET_ALPHA, and MAX_SMOOTH_STEP
  • Battery engagement issues: Explanation of MIN_EFFICIENT_POWER behavior and how to adjust for DC-only setups
  • Marstek app offline status: Clarified that offline/missing readings in the Marstek app is expected for local-only operation and documented how to enable live readings via hame-relay integration

Notes

Documentation-only change; no source code modifications.

https://claude.ai/code/session_013duB5UuF2V583jxfs8T7nB

Summary by CodeRabbit

  • Documentation
    • Expanded FAQ with guidance on configuring signed power sensors in Home Assistant App to prevent drift and oscillation.
    • Added troubleshooting section covering output power oscillation diagnosis, second battery engagement issues, and Marstek app offline behavior.

Review Change Stack

claude added 3 commits May 27, 2026 06:47
Add answers for power oscillation tuning, Shelly vs CT002/CT003 for
multiple batteries, the MIN_EFFICIENT_POWER engagement behavior, the
Marstek app showing the meter offline, and single- vs two-sensor
import/export configuration.

https://claude.ai/code/session_013duB5UuF2V583jxfs8T7nB
THROTTLE_INTERVAL limits how often the source is re-read, while
DEDUPE_TIME_WINDOW drops repeated inbound polls; they are not
interchangeable.

https://claude.ai/code/session_013duB5UuF2V583jxfs8T7nB
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 13acf4dd-afcc-4459-bc23-6ba96dcf7298

📥 Commits

Reviewing files that changed from the base of the PR and between db4b6f7 and 3118753.

📒 Files selected for processing (1)
  • README.md

Walkthrough

The PR expands README.md with FAQ and troubleshooting guidance. It adds a recommendation to use a single signed sensor for import/export power configuration and three new troubleshooting Q&A entries covering power oscillation tuning parameters, battery engagement conditions related to MIN_EFFICIENT_POWER, and Marstek app offline behavior with MQTT configuration notes.

Changes

FAQ and Troubleshooting Documentation

Layer / File(s) Summary
FAQ and troubleshooting documentation updates
README.md
README FAQ section recommends using a single signed sensor for POWER_INPUT_ALIAS/CURRENT_POWER_ENTITY to prevent oscillation. Troubleshooting section adds Q&A entries for diagnosing power oscillation with tuning parameters (THROTTLE_INTERVAL, DEDUPE_TIME_WINDOW, DEADBAND, SMOOTH_TARGET_ALPHA, MAX_SMOOTH_STEP), explaining MIN_EFFICIENT_POWER effects on battery engagement, and clarifying Marstek app offline behavior with MQTT polling via hame-relay (≥ 1.3.5).

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely summarizes the main changes: additions of FAQ entries covering power sensing guidance, multi-battery coordination, and troubleshooting. It directly reflects the primary focus of the documentation-only pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/magical-ritchie-R4jlc

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tomquist tomquist marked this pull request as ready for review May 27, 2026 07:16
@tomquist tomquist merged commit 350bb1c into develop May 27, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants