New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid more device_class lookups for number entities when writing state #102381
Conversation
I missed some more in min/max value
Hey there @home-assistant/core, @Shulyaka, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
@final | ||
def min_value(self) -> float: | ||
"""Return the minimum value.""" | ||
return self._convert_to_state_value(self.native_min_value, floor_decimal) | ||
return self._convert_to_state_value( | ||
self.native_min_value, floor_decimal, self.device_class | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably don't need this but I left it for backwards compat
@final | ||
def max_value(self) -> float: | ||
"""Return the maximum value.""" | ||
return self._convert_to_state_value(self.native_max_value, ceil_decimal) | ||
return self._convert_to_state_value( | ||
self.native_max_value, ceil_decimal, self.device_class | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably don't need this but I left it for backwards compat
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I'm fine with this. it feels lika a bit of a slippery slope to optimize on this small scale though. I think i want some other eyes on it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @bdraco 👍
I agree with Joakim a bit, but then again, these are internal and are allowed to be a bit more complex.
../Frenck
thanks |
Proposed change
I missed some more in min/max value
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
.To help with the load of incoming pull requests: