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

[mqtt.homeassistant] Implement non-deprecated color inference for JSON Schema Lights #17529

Conversation

ccutrer
Copy link
Contributor

@ccutrer ccutrer commented Oct 8, 2024

In particular, use the color_mode attribute to tell us which color space to parse in, instead of trying to guess. Some devices will fill out attributes not-pertinent to the current color mode, and their math might be... less than optimal. Also sync color temp and color channels when the color_mode is the opposite.

…N Schema lights

In particular, use the color_mode attribute to tell us which color space to parse in,
instead of trying to guess. Some devices will fill out attributes not-pertinent to
the current color mode, and their math might be... less than optimal.
Also sync color temp and color channels when the color_mode is the opposite.

Signed-off-by: Cody Cutrer <cody@cutrer.us>
@ccutrer ccutrer requested a review from antroids as a code owner October 8, 2024 20:22
@ccutrer ccutrer requested a review from lsiepel October 9, 2024 21:06
@lsiepel lsiepel added the enhancement An enhancement or new feature for an existing add-on label Oct 10, 2024
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.

Thanks, LGTM

@lsiepel lsiepel merged commit 4cbae6f into openhab:main Oct 11, 2024
5 checks passed
@lsiepel lsiepel added this to the 4.3 milestone Oct 11, 2024
@lsiepel lsiepel changed the title [mqtt.homeassistant] implement non-deprecated color inference for JSON Schema Lights [mqtt.homeassistant] Implement non-deprecated color inference for JSON Schema Lights Oct 11, 2024
@ccutrer ccutrer deleted the mqtt-homeassistant-non-deprecated-json-schema-light-color branch October 11, 2024 12:57
joni1993 pushed a commit to joni1993/openhab-addons that referenced this pull request Oct 15, 2024
…N Schema lights (openhab#17529)

In particular, use the color_mode attribute to tell us which color space to parse in,
instead of trying to guess. Some devices will fill out attributes not-pertinent to
the current color mode, and their math might be... less than optimal.
Also sync color temp and color channels when the color_mode is the opposite.

Signed-off-by: Cody Cutrer <cody@cutrer.us>
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Oct 18, 2024
…N Schema lights (openhab#17529)

In particular, use the color_mode attribute to tell us which color space to parse in,
instead of trying to guess. Some devices will fill out attributes not-pertinent to
the current color mode, and their math might be... less than optimal.
Also sync color temp and color channels when the color_mode is the opposite.

Signed-off-by: Cody Cutrer <cody@cutrer.us>
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.

2 participants