Remove state class from daily net sensors in DSMR Reader#75864
Conversation
|
Hey there @depl0y, mind taking a look at this pull request as it has been labeled with an integration ( |
|
The pull request doesn't fully fix the issue. I just ran the updated code locally for a bit and got this error for another entity: I had code waiting locally that would prevent the sensor from using data if it was only a tiny bit smaller than the previous value, so the total_increasing class could remain. Just removing the state class seems to not have any negative effect on the visualisation of the data, so I guess that's even better. |
Proposed change
This PR fixes entities tracking a net total (and not an increasing total) in the DSMR Reader integration. Fixes warnings like:
integration dsmr_reader has state class total_increasing, but its state is not strictly increasingCausing these not to work with statistics.
Technically, it should be possible to convert these into a
totalstate class with alast_reset; however, these have no value to keep, as this integration already provides sensors with the increasing totals.Type of change
Additional information
Checklist
black --fast homeassistant tests)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest.requirements_all.txt.Updated by running
python3 -m script.gen_requirements_all..coveragerc.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: