Velis Evo2 | Model 4 "sensor.ariston_domestic_hot_water_total_energy_consumption" incorrect & minus readings

[epd5]

Hi,

Relatively new to Home Assistant & would appreciate any help/pointers/insight. Love this integration but having a problem with the ariston_domestic_hot_water_total_energy_consumption sensor.

The cumulative readings aren't adding up to the total consumption per month in the app (However I haven't begun diagnosing this issue as I don't know where to start).

I have made some progress diagnosing a different issue, every now & then I'll get an outlier reading of -1.6 KwH (for example). As this device doesn't contribute anything to the grid this is wrong. Looking through Home Assistant documentation here (https://developers.home-assistant.io/docs/core/entity/sensor/#how-to-choose-state_class-and-last_reset) it looks like the sensor class of the device (state_class=SensorStateClass.TOTAL) should actually be (state_class=SensorStateClass.TOTAL_INCREASING) to prevent this incorrect negative reading. Am I correct in understanding this? Will update the thread if I've found this to solve the issue.

Updated code in line 372 of const.py

AristonSensorEntityDescription(
        key="Domestic hot water total energy consumption",
        name=f"{NAME} domestic hot water total energy consumption",
        icon="mdi:cash",
        entity_category=EntityCategory.DIAGNOSTIC,
#        state_class=SensorStateClass.TOTAL,
        state_class=SensorStateClass.TOTAL_INCREASING,
        device_class=SensorDeviceClass.ENERGY,
        native_unit_of_measurement=UnitOfEnergy.KILO_WATT_HOUR,
        device_features=[
            DeviceFeatures.HAS_METERING,
            ConsumptionType.DOMESTIC_HOT_WATER_TOTAL_ENERGY.name,
        ],
        coordinator=ENERGY_COORDINATOR,
        get_native_value=lambda entity: entity.device.domestic_hot_water_total_energy_consumption,
        get_last_reset=lambda entity: entity.device.consumption_sequence_last_changed_utc,
    ),

[epd5]

Well scratch that, last_reset is on for TOTAL so it shouldn't even be going into negative values. Back to square one.