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

Invert color temperature percentage range #518

Merged
merged 2 commits into from
Oct 10, 2023

Conversation

michikrug
Copy link
Collaborator

@michikrug michikrug commented Oct 10, 2023

As mentioned at https://community.openhab.org/t/oh3-color-light-bulb-and-google-assistant/129938/7, there is some confusion about the relationship between color temperatures and their percentage equivalent.

In the current implementation, the percent values are inverted before conversion to Kelvin. I honestly don't know why this was implemented this way.

To make this more intuitive, we remove the inversion and reflect low kelvin values to low percent values and vice versa.

@michikrug michikrug marked this pull request as ready for review October 10, 2023 17:54
@michikrug michikrug requested a review from marziman as a code owner October 10, 2023 17:54
@michikrug michikrug force-pushed the invert-color-percent branch from 13a289f to 474d86e Compare October 10, 2023 17:55
@michikrug michikrug merged commit 8754c93 into openhab:main Oct 10, 2023
6 checks passed
@michikrug michikrug deleted the invert-color-percent branch October 10, 2023 17:59
@michikrug michikrug added the bug label Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant