Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Warmup.set_override service localize temperature parameter #39

Open
rct opened this issue Feb 16, 2022 · 2 comments
Open

Warmup.set_override service localize temperature parameter #39

rct opened this issue Feb 16, 2022 · 2 comments

Comments

@rct
Copy link
Collaborator

rct commented Feb 16, 2022

On a Home Assistant system configured for Imperial units, the service warmup.set_override needs to be called with a temperature parameter in Celsius, not Fahrenheit.

The example data presented by the UI doesn't specify what units temperature should be in.

Calling the service with a value in Fahrenheit doesn't generate an error, but does appear to be silently rejected because the temperature is out of range.
@artmg
Copy link
Collaborator

artmg commented Feb 24, 2023

This could be investigated and tested as part of #48

@rct
Copy link
Collaborator Author

rct commented Mar 14, 2023

This is probably related to #38. See comments in #38 and #48.

@artmg artmg mentioned this issue Mar 15, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rct @artmg