-
Notifications
You must be signed in to change notification settings - Fork 7
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
Nr of decimals #24
Comments
Hi. I currently see no effect adding decimals to configuration. The node is returning the data as received from Evohome without any influence from the plugin's site. Unfortunately I don't see any possibility for providing this setting back to Evohome. Maybe I'm missing sth.? |
Hmm... That's weird... In domoticz the plugin for the API does provide that option, so it must be there somewhere. The reason I'm asking is that I don't trust the way evohome rounds a bit. It rounds towards the setpoint. With double digits it doesn't seem to do that. Let me have a look if I can find the python for the domoticz api. |
Does this help? Line 61 and beyond: https://github.com/domoticz/domoticz/blob/development/hardware/EvohomeWeb.cpp |
Not really. Can’t imagine that line 1935 and following would do what we try to get working. But this might be some kind of unclear to me. |
Ah, found out there's a second part to the plugin... C++, so no clue how it works: |
Looking at the HA integration i found the following:
https://www.home-assistant.io/integrations/evohome/ Exactly what i meant :) |
I see. But you didn‘t mention that there would be a call to an old API to get those values. I think this is a risk for fetching that data because we‘ve to expect that this old API might be deleted in future. Isn’t it? |
Nevertheless I‘m not sure how to implement for now. |
Tbh; i didn't know it was a hybrid API till i just found out this. And yes it's a risk, for which you can probably build a function "If value from V1 available then value from V1, otherwise V2'. I really dont understand the absurd way evohome is rounding is also the way the API data works. Totally rediculous. |
Hi!
Would it be possible to add the number of decimals to the configuration?
Evohome supports two decimals. Would be nice to have them available.
Currently it's only half degrees.
Br,
Michael
The text was updated successfully, but these errors were encountered: