Energy Dashboard configuration about homeassistant-fusionsolar

[GiorgioUghini]

Every midnight, while other statistics resets to zero (in the frontend, in the backend they are still counting energy), the two battery statistics reset to negative values (in the frontend, in the backend they are reset to zero as they are daily stats).

In HA documentation they say that they keep the initial value when the statistic is added, as zero-point... Don't know if this may be relevant?

[tijsverkoyen]

I think I will first need to read up on the documentation.
I don't think the Huawei Fusion Solar API exposes when the data is reset, for probably I will need to recalculate of take the current time or, ...

I don't have much time the coming weeks as I'm busy with other projects. So don't expect a change very soon.
Feel free to create a pull request yourself.

[GiorgioUghini]

Actually, Home Assistant recognize the "last reset" by looking if a "total increasing" sensor goes to zero. At the end the logic is clear: the sensor starts from zero, it is total and it is INCREASING, so it goes to (say) 20kWh, and if this sensor drops to 0, well that's a reset. HA is intelligent enough for this.
Problem is that there's something in the initialization phase that set the so-called "zero-value" of the sensor to the value that it has in that moment. In this way, when it resets, it goes back to the zero-value (so if you setup this integration at 4PM when you already produced 15kWh, every middnight, the dashboard will display -15kWh). Calculation is: actual value (0) - zerovalue (15) = -15.
I need to evaluate a possible solution for this too.
There's no hurry, your work is already excellent as is. Just putting the issue here if anyone would like to contribute. I will, as soon as I manage a possible workaround for this.

[GiorgioUghini]

Progress so far:

1. Battery discharge capacity and Battery charge capacity are set to total, while they can set to total_increasing. See attached image. If sensor are kept total (without increasing) we should put in place routines to set last_reset based on drops to 0.
   

@tijsverkoyen for this specific behavior (battery charge and battery discharge capacities) state_class to total_increasing will solve the issue. But, we should be able to make this reasoning for all statistics, for example, the daily_* or monthly_* entities provided by Huawei.

I'm guessing if I'm the only one with this behavior (or maybe I'm the only one with a battery tracked in HA energy dashboard?)

[GiorgioUghini]

Changing line 535 in sensor.py from FusionSolarRealtimeDeviceDataEnergySensor to FusionSolarRealtimeDeviceDataEnergyTotalIncreasingSensor would solve.
But, I'm not sure if doing so would break other things. It shouldn't, but who knows. I'm testing it locally on my HA istance - keep you posted.

[tijsverkoyen]

I have merged your PR.
I will create a PR in a few moments