Move onewire raw_value attribute to separate sensor#61093
Move onewire raw_value attribute to separate sensor#61093epenet wants to merge 14 commits intohome-assistant:devfrom
Conversation
|
Hey there @garbled1, mind taking a look at this pull request as it has been labeled with an integration ( |
|
Could this be included in 2021.12.0 ? |
|
I worry that this will break a certain class of automation. We don't know if users are using this in automations anywhere. While the temperature of my back yard is not useful at 3 decimal places, the temperature of my aquarium, or anything I'm attempting to temperature control, absolutely is. Or if I'm using a pair of sensors to measure a minor difference between two things, such as pressure sensors on either side of an AC filter to measure the filter dirt. I fully get that alot of people probably use these to just measure a few room temperatures here and there, and that is the sort of default case, but I'd hate to break anyone doing anything more advanced. |
As discussed on Discord, I will use an option flow instead to deactivate the state attribute. |
|
Another option would to be to make the raw value its own sensor and disable it by default |
There was a problem hiding this comment.
Don't clone the raw value of a moisture device, they are identical.
There was a problem hiding this comment.
same with 0-3, and the leaf wetness too.
b20985f to
cf4b935
Compare
|
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
|
Keeping as draft until #63222 is resolved |
|
Note: I have opened an arch discussion to see if there are other ways to solve this: home-assistant/architecture#713 |
|
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
|
Not stale |
|
Giving up on this - probably not really needed. |
Breaking change
The
raw_valuestate attribute has been removed from all onewire entities.Please use the corresponding sensor instead.
Proposed change
The
raw_valuestate attribute has been removed from all onewire entities.In the case of
temperature,pressure,illuminanceandhumiditysensors, this has been moved to a separate sensor.For the other sensors (
voltage,count,moisture) the automatic rounding has been removedType 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: