Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Solax Remaining Battery Capacity state class #1139

Conversation

prehor
Copy link
Contributor

@prehor prehor commented Nov 19, 2024

The state class for the Remaining Battery Capacity should be removed, consistent with other battery capacity sensors using the ENERGY device class. The ENERGY device class cannot use the MEASUREMENT state class.

Fixies #1137

The state class for the Remaining Battery Capacity should be removed, consistent with other battery capacity sensors using the ENERGY device class. The ENERGY device class cannot use the MEASUREMENT state class.
@prehor
Copy link
Contributor Author

prehor commented Nov 19, 2024

A better solution might be to change the device class to ENERGY_STORAGE, which is compatible with the MEASUREMENT state class.

@prehor prehor closed this Nov 19, 2024
@steps56
Copy link

steps56 commented Nov 26, 2024

plugin_solax.zip
Hi,
as I already told at my initial issue post:
The clean solution is to change
state class
from
ENERGY
to:
TOTAL

@steps56
Copy link

steps56 commented Nov 26, 2024

plugin_solax.zip

The state class for the Remaining Battery Capacity should be removed, consistent with other battery capacity sensors using the ENERGY device class. The ENERGY device class cannot use the MEASUREMENT state class.

Fixies #1137

Isn't any solution, as still throwing error/warning at system logs.
Added edited errorfree solax.py as a .zip at top

@prehor prehor deleted the remove-solax-remaining-battery-capacity-state-class branch December 4, 2024 21:00
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