-
Notifications
You must be signed in to change notification settings - Fork 11
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
Live Consumption using the kW instead of kWh #24
Comments
Hmm, let me look into this |
any update on this? i'm sill getting the errors |
I had an initial look - I think in hindsight that although this is the proper metric - it's actually a breaking change. Going to look into a bit more soon |
Ok - I have looked into this a bit more. The last release as mentioned previously is a semi-breaking change (but might be recoverable) The live consumption sensor, is using If you go to: https://my.home-assistant.io/redirect/developer_statistics you can click "Fix issue" and remediate past values to be Let me know how you go with this - and I can update the README.md for this. Apologies for the impact! |
I went to do this but got the following choice which worried me about the scaling between kWh to W. Shouldn't it be from kWh to "kW" if there is only a unit change? "Update the unit of the historic statistic values from 'kWh' to 'W', without converting." |
Yeah, i did "Update the unit of the historic statistic values from 'kWh' to 'W', without converting." and then used powerpal_total_consumption going forward
|
Yep, I think thats right from memory. |
Version of the custom_component: 0.3.0
Describe the bug
It looks like in the latest update, we've accidentally changed the PowerpalLiveConsumptionSensor to use
UnitOfPower.KILO_WATT
instead ofUnitOfEnergy.KILO_WATT_HOUR
hass-powerpal/custom_components/powerpal/sensor.py
Line 114 in 88d90a7
This has caused the below error, and home assistant is now no longer calculating statistics.
As stated in a previous issue, live consumption should be WattHours, not Watts.
Does that seem correct? or am i missing something here.
Debug log
The text was updated successfully, but these errors were encountered: