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

[Bug]: v1.40A2 - State Class and Device Class Differs Between Batteries #684

Closed
5 tasks done
Roving-Ronin opened this issue Apr 16, 2024 · 1 comment
Closed
5 tasks done
Labels
bug Something isn't working

Comments

@Roving-Ronin
Copy link
Contributor

Describe the issue

Is this correct, as the existing 'battery' sensor that shows the total of all batteries has a state class of TOTAL, and device class of ENERGY. Whilst the new battery_1_x type sensors for the same data, have the state class of MEASUREMENT and the device class of ENERGY_STORAGE (which is correct for a battery)

image

Works either way, but flagging in case the existing sensor.battery_ sensors need to be updated to the newer state/device classes?

Bescribe your Huawei Solar Setup

Inverter Type: SUN2000-5KTL-L1 (Qty 2)
Inverter Firmware version: V200R001C00SPC138
SDongle present: yes - V200R022C10SPC108
Power meter present: single phase
Battery: LUNA2000 10kWh
Battery Firmware version: V100R002C00SPC125

How do you connect to the inverter?

Via the SDongle, wireless connection

Upload your Diagnostics File

N/A.

Upload your relevant debug logs

N/A.

Please confirm the following:

  • I'm running the latest release of Home Assistant.
  • I'm running the latest release of this integration.
  • I did not find an existing issue describing this problem.
  • I did upload the diagnostics-file that I could retrieve from the 'Devices & Services Page'
  • I did upload the relevant debug logs (via 'Enable Debug Logging'-feature or by manually configuring HA logging)
@Roving-Ronin Roving-Ronin added the bug Something isn't working label Apr 16, 2024
@wlcrs wlcrs closed this as completed in d4816a0 Apr 16, 2024
@wlcrs
Copy link
Owner

wlcrs commented Apr 16, 2024

Thank you for noticing those two flaws. I've changed them all to "total" and "energy_storage".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants