Incorrect Input/Output values? kWh instead of Wh?

[AshleyJackson]

Hello,

I own a Delta Pro and within the energy section of Home Assistant it is showing kWh generated and used.

Is this value being converted incorrectly by the Integration?

Please let me know if you need any test data from me.
Thanks.

[knyghtryda]

@AshleyJackson @tolwi Just a reminder that first day integrations will always be strange, because the value stored is continually increasing. That means the first time it gets picked up It will read your all time total. After a day it will start to display your delta value, not cumulative value. Here are my day and week values:

Note that due to some logging issues I had yesterday the daily solar in the second graph is kinda bugged out. It looks like it grabbed the cumulative value again for some reason.

[tolwi]

hm, good point about initial accumulated values ...
can you please attach one more screnshot with pd.chgSunPower history for like 7 days

[knyghtryda]

Here's the last few days of Solar In Energy

And Solar Power

[knyghtryda]

@tolwi I figured out the energy spike issue, though I'm not exactly sure how to solve it. What seems to be happening is that the Ecoflow will occasionally report a zero value for the energy. It then goes back to reporting the normal energy value. That just confuses the crap out of the Energy Dashboard which thinks it just went from a baseline of zero to full cumulative. This probably means we need to either not report zero values at all, or take the previous value and use it if a value comes in at zero. I'm not familiar enough with HA to know quite how to do either of those things though.

Here's the zero spikes

Here's the energy spikes

[tolwi]

def _update_value(self, val: Any) -> bool:
        ival = int(val)
        if ival > 0:
          return super()._update_value(ival)
        else:
          return False

with 0 initial value this logic should be valid for all SensorStateClass.TOTAL_INCREASING sensors

[knyghtryda]

@tolwi This is where i put your code but I'm still seeing the value go wrong after a restart.

class EnergySensorEntity(BaseSensorEntity):
    _attr_device_class = SensorDeviceClass.ENERGY
    _attr_native_unit_of_measurement = UnitOfEnergy.WATT_HOUR
    _attr_state_class = SensorStateClass.TOTAL_INCREASING
    _attr_native_value = 0
    def _update_value(self, val: Any) -> bool:
        ival = int(val)
        if ival > 0:
          return super()._update_value(ival)
        else:
          return False

[tolwi]

try to remove _attr_native_value = 0

[AshleyJackson]

I've now gotten this twice today.
First time was when I restarted by Home Assistant server, second time was when I upgraded to v0.11.1.

[knyghtryda]

I've implemented @tolwi 's fix locally. I'll report here if it works, and then hopefully create a PR for it.

[vRzn]

i have a river pro, but it looks like there is no cumulative (Wh) value out of that device.
is it available for that device, or is it just select (larger) devices that have it?
i would love to add my river pro to my dashboard

[PaSch1]

You can use the helper "Riemann Summenintegralsensor" for this.