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

Added LUT profiles for 2x Signify + 3x Ikea models #47

Merged
merged 9 commits into from
Jul 3, 2021
Merged

Added LUT profiles for 2x Signify + 3x Ikea models #47

merged 9 commits into from
Jul 3, 2021

Conversation

nepozs
Copy link
Contributor

@nepozs nepozs commented Jul 2, 2021

additionally created mappings for future measurements (Jiawen brand and 1 Ikea model)

nepozs and others added 9 commits June 28, 2021 14:23
used new model mappings in case LED1649C5
New directory names in format LEDxxxxyzz, as printed on bulbs and boxes.
+ directory mapping for Jiawen brand
Signify LLC011 has only H/S mode (by design - it is RGB only)
due to power outage I've had to change measure plans - instead of RGBW model I've prepared last white spectrum model
@bramstroker
Copy link
Owner

Thanks, great job!

@bramstroker bramstroker merged commit db901b0 into bramstroker:master Jul 3, 2021
@bramstroker
Copy link
Owner

Btw I see color_temp is missing for LLC011. When the light is controllable by color it should have both a hs file and color_temp file, because you can also set the light to color temperature mode in home assistant.
Could you create that one please?
When it is included I will release version 0.2.5 containing your new profiles.
Thanks alot for your work on these.

@bramstroker
Copy link
Owner

bramstroker commented Jul 10, 2021

@nepozs did you see my comment above about the missing lut profile for LLC011. Are you able to take measurements for that one in the coming days? Than I can release a new version containing the 5 new lights you worked on.

@nepozs
Copy link
Contributor Author

nepozs commented Jul 11, 2021

I have missed your comment, but LLC011 does not support color_temp mode (I was sure that I wrote it in PR) so it is OK I think.

@bramstroker
Copy link
Owner

@nepozs ok awesome! Was very busy last days. Will make the release this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants