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

[UoM] Absolute vs. Relative Temperature #855

Closed
keilw opened this issue Jun 5, 2019 · 3 comments
Closed

[UoM] Absolute vs. Relative Temperature #855

keilw opened this issue Jun 5, 2019 · 3 comments
Labels
UoM Units of Measurement

Comments

@keilw
Copy link

keilw commented Jun 5, 2019

Let me try this here now.
There was a discussion prior to unitsofmeasurement/indriya#128 around absolute/relative Temperature values and how SmartHome should handle them. So far this was done in an application-specific way. JSR 385 (Units of Measurement 2.0) does provide a Quantity Scale that can be either ABSOLUTE or RELATIVE, but no implementation-specific operations are applied yet. It is not so easy, because the type Temperature cannot be kept hold of in Java due to Generics restrictions.
What was done in SmartHome or OpenHAB so far, and is there a branch or class we could use as inspiration for something that might work in a more general purpose manner? Or do you feel it's better to leave it up to individual solutions to react based on the new scale?

@wborn
Copy link
Member

wborn commented Jun 5, 2019

Looks like the discussion for this was in eclipse-archived/smarthome#5697 and @triller-telekom made some workarounds in eclipse-archived/smarthome#5679.

@wborn wborn added the UoM Units of Measurement label Jun 5, 2019
@keilw
Copy link
Author

keilw commented Jun 6, 2019

Thanks, we'll check it out for the RI.

Rosi2143 pushed a commit to Rosi2143/openhab-core that referenced this issue Dec 26, 2020
This is a new feature in ESH, added with PR #6777.

Signed-off-by: Florian <florian.schmidt.welzow@t-online.de>
@wborn
Copy link
Member

wborn commented Oct 13, 2021

Let's close this since we also switched to using indriya (#2319).
The temperature implementation changes did cause a few minor issues and inconveniences for some (#2386).

@wborn wborn closed this as completed Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UoM Units of Measurement
Projects
None yet
Development

No branches or pull requests

2 participants