Skip to content

[solarman] deye_hybrid definition: deduplicate A6 register - #21409

Open
dilyanpalauzov wants to merge 1 commit into
openhab:mainfrom
dilyanpalauzov:solarman_deduplicate_a6
Open

[solarman] deye_hybrid definition: deduplicate A6 register#21409
dilyanpalauzov wants to merge 1 commit into
openhab:mainfrom
dilyanpalauzov:solarman_deduplicate_a6

Conversation

@dilyanpalauzov

Copy link
Copy Markdown
Contributor

Solarman logger is a wrapper around Modbus. The A6 register cannot stand at the same time for “Micro-inverter Power”, “Gen-connected Status” and “Gen Power”.

According to https://marklabs.pl/en/deye-modbus-complete-register-list/ for the same register different Deye models provide different information. Deye Modbus registers are described at https://github.com/kbialek/deye-inverter-mqtt/blob/main/README.md . “Deye SG01LP1” is “deye_hybrid” and https://github.com/kbialek/deye-inverter-mqtt/blob/main/docs/metric_group_deye_hybrid.md says A6 register is “Micro-inverter Power”. Therefore the A6 register is neither “Gen-connected Status” nor “Gen Power”.

However I have no idea what “Micro-inverter Power” means for an inverter, which is not microinverter.

Utilizing A6 for different purpose should be done by providing a different YAML device definition.

Solarman logger is a wrapper around Modbus. The A6 register cannot stand at the
same time for “Micro-inverter Power”, “Gen-connected Status” and “Gen Power”.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the Solarman binding’s deye_hybrid device definition to avoid mapping the same Modbus register (0x00A6) to multiple, conflicting metrics, aligning the definition with model-specific register semantics.

Changes:

  • Removed the Gen-connected Status metric that previously read from register 0x00A6.
  • Removed the Gen Power metric that previously read from register 0x00A6.
  • Left 0x00A6 mapped only to Micro-inverter Power in deye_hybrid.yaml.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@dilyanpalauzov

Copy link
Copy Markdown
Contributor Author

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