HomeWizard sensor.p1_meter_total_power_import_t1 not strictly increasing

tjorim commented 1 year ago

The problem

Once in a while (the past week or two) I get warnings that total_power_import_t1 has state class total_increasing, but its state is not strictly increasing. Triggered by state 228.57 (228.57) with last_updated set to 2023-03-02T06:44:53.372421+00:00.

What version of Home Assistant Core has the issue?

core-2023.3.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

HomeWizard Energy

Link to integration documentation on our website

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

Diagnostics information

config_entry-homewizard-b0d259ef94b86c52e40c1eef3106f477.json.txt

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.sensor.recorder
Source: components/sensor/recorder.py:335
Integration: Sensor (documentation, issues)
First occurred: 7:45:08 AM (1 occurrences)
Last logged: 7:45:08 AM

Entity sensor.p1_meter_total_power_import_t1 from integration homewizard has state class total_increasing, but its state is not strictly increasing. Triggered by state 228.57 (228.57) with last_updated set to 2023-03-02T06:44:53.372421+00:00. Please create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+homewizard%22

Additional information

No response

home-assistant[bot] commented 1 year ago

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

Code owner commands

Code owners of `homewizard` 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 homewizard` Removes the current integration label and assignees on the issue, add the integration domain after the command.

_{^{(message by CodeOwnersMention)}}

homewizard documentation homewizard source _{^{(message by IssueLinks)}}

DCSBL commented 1 year ago

Thank you for opening this issue.

The message is added in https://github.com/home-assistant/core/pull/55197. It indicates that the sensor value decreased by less then 10%. This is logged once, so it may occur more often. It should not happen as the 'total' is total_increasing.

Can you check in your log or history to see what happens around that moment with the sensor value?

tjorim commented 1 year ago

I went through the logbook and history, did not see anything weird. I turned on debug logging and hope to catch it again tomorrow.

tjorim commented 1 year ago

Already happened again, attached the debug logs:

Logger: homeassistant.components.sensor.recorder
Source: components/sensor/recorder.py:335
Integration: Sensor (documentation, issues)
First occurred: 10:24:16 AM (1 occurrences)
Last logged: 10:24:16 AM

Entity sensor.p1_meter_total_power_import_t1 from integration homewizard has state class total_increasing, but its state is not strictly increasing. Triggered by state 228.87 (228.87) with last_updated set to 2023-03-02T09:24:11.388494+00:00. Please create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+homewizard%22

Edit: I snipped out the important part and censored some stuff:

2023-03-02 10:24:11.154 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api, None
2023-03-02 10:24:11.183 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"product_name":"P1 meter","product_type":"HWE-P1","serial":"5C2FAF000000","firmware_version":"4.14","api_version":"v1"}
2023-03-02 10:24:11.184 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api/v1/data, None
2023-03-02 10:24:11.324 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"wifi_ssid":"Proximus-Home-0000","wifi_strength":34,"smr_version":50,"meter_model":"Fluvius 123456789_A","unique_id":"**censored**","active_tariff":1,"total_power_import_kwh":548.807,"total_power_import_t1_kwh":228.866,"total_power_import_t2_kwh":319.941,"total_power_export_kwh":0,"total_power_export_t1_kwh":0,"total_power_export_t2_kwh":0,"active_power_w":141,"active_power_l1_w":141,"active_voltage_l1_v":235.7,"active_current_l1_a":0.8,"active_power_average_w":89,"montly_power_peak_w":1754,"montly_power_peak_timestamp":230301180000,"total_gas_m3":107.027,"gas_timestamp":230302102328,"gas_unique_id":"**censored**","external":[{"unique_id":"**censored**","type":"gas_meter","timestamp":230302102328,"value":107.027,"unit":"m3"}]}
2023-03-02 10:24:11.327 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api/v1/system, None
2023-03-02 10:24:11.385 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"cloud_enabled":true}
2023-03-02 10:24:11.386 DEBUG (MainThread) [homeassistant.components.homewizard.coordinator] Finished fetching homewizard data in 0.232 seconds (success: True)
2023-03-02 10:24:16.156 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api, None
2023-03-02 10:24:16.179 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"product_name":"P1 meter","product_type":"HWE-P1","serial":"5C2FAF000000","firmware_version":"4.14","api_version":"v1"}
2023-03-02 10:24:16.179 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api/v1/data, None
2023-03-02 10:24:16.312 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"wifi_ssid":"Proximus-Home-0000","wifi_strength":32,"smr_version":50,"meter_model":"Fluvius 123456789_A","unique_id":"**censored**","active_tariff":1,"total_power_import_kwh":548.807,"total_power_import_t1_kwh":228.866,"total_power_import_t2_kwh":319.941,"total_power_export_kwh":0,"total_power_export_t1_kwh":0,"total_power_export_t2_kwh":0,"active_power_w":140,"active_power_l1_w":140,"active_voltage_l1_v":235.3,"active_current_l1_a":0.8,"active_power_average_w":90,"montly_power_peak_w":1754,"montly_power_peak_timestamp":230301180000,"total_gas_m3":107.027,"gas_timestamp":230302102328,"gas_unique_id":"**censored**","external":[{"unique_id":"**censored**","type":"gas_meter","timestamp":230302102328,"value":107.027,"unit":"m3"}]}
2023-03-02 10:24:16.316 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api/v1/system, None
2023-03-02 10:24:16.376 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"cloud_enabled":true}
2023-03-02 10:24:16.377 DEBUG (MainThread) [homeassistant.components.homewizard.coordinator] Finished fetching homewizard data in 0.221 seconds (success: True)
2023-03-02 10:24:16.382 WARNING (MainThread) [homeassistant.components.sensor.recorder] Entity sensor.p1_meter_total_power_import_t1 from integration homewizard has state class total_increasing, but its state is not strictly increasing. Triggered by state 228.87 (228.87) with last_updated set to 2023-03-02T09:24:11.388494+00:00. Please create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+homewizard%22
2023-03-02 10:24:21.155 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api, None
2023-03-02 10:24:21.177 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"product_name":"P1 meter","product_type":"HWE-P1","serial":"5C2FAF000000","firmware_version":"4.14","api_version":"v1"}
2023-03-02 10:24:21.178 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api/v1/data, None
2023-03-02 10:24:21.308 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"wifi_ssid":"Proximus-Home-0000","wifi_strength":34,"smr_version":50,"meter_model":"Fluvius 123456789_A","unique_id":"**censored**","active_tariff":1,"total_power_import_kwh":548.807,"total_power_import_t1_kwh":228.866,"total_power_import_t2_kwh":319.941,"total_power_export_kwh":0,"total_power_export_t1_kwh":0,"total_power_export_t2_kwh":0,"active_power_w":142,"active_power_l1_w":142,"active_voltage_l1_v":236.7,"active_current_l1_a":0.8,"active_power_average_w":91,"montly_power_peak_w":1754,"montly_power_peak_timestamp":230301180000,"total_gas_m3":107.027,"gas_timestamp":230302102328,"gas_unique_id":"**censored**","external":[{"unique_id":"**censored**","type":"gas_meter","timestamp":230302102328,"value":107.027,"unit":"m3"}]}
2023-03-02 10:24:21.312 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api/v1/system, None
2023-03-02 10:24:21.374 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"cloud_enabled":true}
2023-03-02 10:24:21.375 DEBUG (MainThread) [homeassistant.components.homewizard.coordinator] Finished fetching homewizard data in 0.220 seconds (success: True)
2023-03-02 10:24:26.154 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api, None
2023-03-02 10:24:26.190 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"product_name":"P1 meter","product_type":"HWE-P1","serial":"5C2FAF000000","firmware_version":"4.14","api_version":"v1"}
2023-03-02 10:24:26.191 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api/v1/data, None
2023-03-02 10:24:26.326 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"wifi_ssid":"Proximus-Home-0000","wifi_strength":36,"smr_version":50,"meter_model":"Fluvius 123456789_A","unique_id":"**censored**","active_tariff":1,"total_power_import_kwh":548.807,"total_power_import_t1_kwh":228.866,"total_power_import_t2_kwh":319.941,"total_power_export_kwh":0,"total_power_export_t1_kwh":0,"total_power_export_t2_kwh":0,"active_power_w":141,"active_power_l1_w":141,"active_voltage_l1_v":236.3,"active_current_l1_a":0.8,"active_power_average_w":92,"montly_power_peak_w":1754,"montly_power_peak_timestamp":230301180000,"total_gas_m3":107.027,"gas_timestamp":230302102328,"gas_unique_id":"**censored**","external":[{"unique_id":"**censored**","type":"gas_meter","timestamp":230302102328,"value":107.027,"unit":"m3"}]}
2023-03-02 10:24:26.331 DEBUG (MainThread) [homewizard_energy.homewizard_energy] GET, http://192.168.1.104/api/v1/system, None
2023-03-02 10:24:26.399 DEBUG (MainThread) [homewizard_energy.homewizard_energy] 200, {"cloud_enabled":true}
2023-03-02 10:24:26.399 DEBUG (MainThread) [homeassistant.components.homewizard.coordinator] Finished fetching homewizard data in 0.246 seconds (success: True)

DCSBL commented 1 year ago

The values do not change at all around that moment..?

Erhm @emontnemery, as you have designed the behaviour for this log message, can you maybe take a look at this?

issue-triage-workflows[bot] commented 1 year ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

home-assistant / core