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.57k stars 30.74k forks source link

Daikin daily energy consumption sensor is now increasing and not reseting to 0 anymore #121821

Open tofuSCHNITZEL opened 4 months ago

tofuSCHNITZEL commented 4 months ago

The problem

Before the 1st of July the sensor "Compressor energy consumption" did reset to 0 daily (hence its internal key was chosen to be "ATTR_TOTAL_ENERGY_TODAY") but it seems since I guess the update to HA 2024.7 it is continuously increasing...

image

I would not say it is an issue per se but is this now a new "feature" of the history/statistics? I checked the used pydaikin lib but I did not see any changes to the code that would explain this change in the sensor (so I surmised it is a change in how HA calculates the statistics?)

What version of Home Assistant Core has the issue?

core-2024.7.0

What was the last working version of Home Assistant Core?

core-2024.6.0

What type of installation are you running?

Home Assistant OS

Integration causing the issue

daikin

Link to integration documentation on our website

https://www.home-assistant.io/integrations/daikin/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 4 months ago

Hey there @fredrike, mind taking a look at this issue as it has been labeled with an integration (daikin) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `daikin` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign daikin` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


daikin documentation daikin source (message by IssueLinks)

fredrike commented 2 months ago

@tofuSCHNITZEL I would guess that this an issue due to wrong setting in last reset, https://developers.home-assistant.io/docs/core/entity/sensor/#properties, can you show the attributes of the energy sensor.

tofuSCHNITZEL commented 2 months ago

image is this what you need?

fredrike commented 2 months ago

image is this what you need?

Yes, this confirms that it has changed to state_class: total_increasing. I guess this is the expected behaviour as it doesn't reset.

I'll close this.

fredrike commented 2 months ago

@home-assistant close

tofuSCHNITZEL commented 2 months ago

But then maybe we should rename the sensor internally to something else than "today" or does it still reset every day but just home assistant does not display it like that anymore?

fredrike commented 2 months ago

Yes you are right. It should be renamed, it just reflect the daily values reported by the Daikin device.

fredrike commented 2 months ago

@home-assistant reopen

fredrike commented 2 months ago

Assign @mlemainque