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.

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

[ecobee] Increase temperature precision to decimal #14517

Merged
merged 2 commits into from
Mar 2, 2023

Conversation

darthbevis
Copy link
Contributor

@darthbevis darthbevis commented Mar 1, 2023

Correcting rounding of setHold values so we have 0.1°F tolerance instead of 1°F. Makes it easier to use in °C

Signed-off-by: Bevan Watkiss darthbevis@gmail.com

@darthbevis darthbevis requested a review from mhilbush as a code owner March 1, 2023 02:21
Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@mhilbush mhilbush left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just have one question about the conversion truncating vs rounding up/down.

Copy link
Contributor

@mhilbush mhilbush left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lsiepel
Copy link
Contributor

lsiepel commented Mar 1, 2023

You still have to apply spotless, to fix the failing build.

@jlaur
Copy link
Contributor

jlaur commented Mar 1, 2023

You still have to apply spotless, to fix the failing build.

When spotless is applied and DCO is fixed, we can merge. @darthbevis, can you also rename the PR title to summarize the user-experienced improvement in a few words?

@jlaur jlaur added the enhancement An enhancement or new feature for an existing add-on label Mar 1, 2023
Correcting rounding of setHold values so we have 0.1°F tolerance instead of 1°F.  Makes it easier to use in °C

Signed-off-by: darthbevis <darthbevis@gmail.com>
Signed-off-by: darthbevis <darthbevis@gmail.com>
@darthbevis darthbevis force-pushed the setHoldRoundingFix branch from e195499 to 8026ca5 Compare March 2, 2023 13:58
@darthbevis darthbevis changed the title [ecobee] Update EcobeeUtils.java to round on a 1/10th degree instead of 1 degree Fahrenheit when converting to ecobee units [ecobee] setHold truncates to 1*F, causing use in *C values to not match desired value. Update EcobeeUtils.java to truncate on a 1/10th degree instead of 1 degree Fahrenheit when converting to ecobee units Mar 2, 2023
@lsiepel
Copy link
Contributor

lsiepel commented Mar 2, 2023

May i suggest a hsorter title for the release notes?:
[ecobee] Increase temperature precision to decimal

@darthbevis darthbevis changed the title [ecobee] setHold truncates to 1*F, causing use in *C values to not match desired value. Update EcobeeUtils.java to truncate on a 1/10th degree instead of 1 degree Fahrenheit when converting to ecobee units [ecobee] Increase temperature precision to decimal Mar 2, 2023
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jlaur jlaur merged commit 660ec10 into openhab:main Mar 2, 2023
@jlaur jlaur added this to the 4.0 milestone Mar 2, 2023
miloit pushed a commit to miloit/openhab-addons that referenced this pull request Mar 10, 2023
* Update EcobeeUtils.java

Correcting rounding of setHold values so we have 0.1°F tolerance instead of 1°F.  Makes it easier to use in °C

Signed-off-by: darthbevis <darthbevis@gmail.com>
Signed-off-by: miloit <MichaelLoercher@web.de>
miloit pushed a commit to miloit/openhab-addons that referenced this pull request Mar 10, 2023
* Update EcobeeUtils.java

Correcting rounding of setHold values so we have 0.1°F tolerance instead of 1°F.  Makes it easier to use in °C

Signed-off-by: darthbevis <darthbevis@gmail.com>
Signed-off-by: miloit <MichaelLoercher@web.de>
renescherer pushed a commit to renescherer/openhab-addons that referenced this pull request Mar 23, 2023
* Update EcobeeUtils.java

Correcting rounding of setHold values so we have 0.1°F tolerance instead of 1°F.  Makes it easier to use in °C

Signed-off-by: darthbevis <darthbevis@gmail.com>
FordPrfkt pushed a commit to FordPrfkt/openhab-addons that referenced this pull request Apr 20, 2023
* Update EcobeeUtils.java

Correcting rounding of setHold values so we have 0.1°F tolerance instead of 1°F.  Makes it easier to use in °C

Signed-off-by: darthbevis <darthbevis@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants