search

home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.61k stars 30.77k forks source link

Reopen - Reactive power energy units #98453 #126646

Closed mark-carline closed 1 month ago

mark-carline commented 1 month ago 
          The architectural discussion has not yet been approved. Going to close this PR in the mean time awaiting the approval.

../Frenck

Originally posted by @frenck in https://github.com/home-assistant/core/issues/98453#issuecomment-1801813166

mark-carline commented 1 month ago

Reopening this issue: https://github.com/home-assistant/core/issues/85507

Need to update this to include reactive_energy with "varh" and "kvarh"

Also: SensorDeviceClass.REACTIVE_POWER only has "var" and should also have "kvar" SensorDeviceClass.APPARENT_POWER only has "VA" and should also have "kVA"

Sorry not sure if I have opened this issue correctly.

Thanks

mib1185 commented 1 month ago

please use the issue template, it is on purpose, to help us to help you 😉

mark-carline commented 1 month ago

Ok - Please feel to close. This is a new feature so i went via the process and opened forum request:

https://community.home-assistant.io/t/can-we-please-add-device-class-reactive-energy-in-sensor/774319

Thanks