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

Continue to adapt color temperature down to the sleep temperature after sunset #87

Merged
merged 31 commits into from
Apr 3, 2023
Merged

Conversation

igiannakas
Copy link
Contributor

Continue to adapt color temperature down to the sleep temperature after sunset. Results in a gradually warming light during the night time rather than a fixed color temperature throughout the night time.

Continue to adapt color temperature down to the sleep temperature after sunset. Results in a gradually warming light during the night time rather than a fixed color temperature throughout the night time.
@igiannakas igiannakas changed the title Update switch.py Continue to adapt color temperature down to the sleep temperature after sunset Feb 9, 2021
@basnijholt
Copy link
Owner

basnijholt commented Feb 9, 2021

This looks good! Have you been testing this at home yourself for sometime?

@igiannakas
Copy link
Contributor Author

This looks good! Have you been testing this at home yourself for sometime?

Yeap I’ve got it setup in my house across 4 zones and works like a charm. Try it out, I’ve set a 4500k max temp, 3000k minimum colour temp and a 2000 sleep temp.

See below graph. The blips are due to me switching on and off the adaptive light switch.

8CBE3CEA-3560-4D6E-A4FE-3316F2E8ED07

@montezumeflex
Copy link

Have been using your integration for a few months now, this looks like a very useful addition. @basnijholt do you think you will merge this into next release?

@basnijholt
Copy link
Owner

Unfortunately, I haven't had the time to test it myself yet.

I know many people will blindly smash the update button in HACS so I am a bit careful with changing the core logic.

@montezumeflex
Copy link

I understand, your point is very valid. Just want to say thanks again, been running your integration for the last few months and it is already amazing in its current form. Looking forward where you will take it in the future.

@igiannakas
Copy link
Contributor Author

Thought to comment back- been using it and has been flawlessly since the beginning of Feb.

@kmalinich
Copy link

I really like this idea, a lot. Adaptive Lighting makes everything so smooth you never notice the lighting changes - it's just always perfect. Except when transitioning to sleep mode, which if it's a big change, can be jarring.

Some food for thought:
I use the sleep mode a bit differently than others, I think.
For most of my lights, I have a 2500K-6536K color temp range, with 25%-100% brightness range for the normal Adaptive Lighting operation.
For sleep mode, I have it set to 4500K, 1% brightness. It creates a pleasant moonlight effect, great for walking around at night after you've been sleeping, since it's barely any light - but it's a cool-ish white (like the moon). For some of my fixtures that have bulbs without diffusers, you can even get crisp shadows that look just like moonlight shadows. I love it.

My point is - the functionality in the PR seems like it kind of assumes the sleep color temp is lower (on the kelvin scale) than the min color temp, and that's not how all users use the sleep settings.

@silverspringhusker
Copy link

silverspringhusker commented Dec 15, 2021

Some food for thought: I use the sleep mode a bit differently than others, I think. For most of my lights, I have a 2500K-6536K color temp range, with 25%-100% brightness range for the normal Adaptive Lighting operation. For sleep mode, I have it set to 4500K, 1% brightness. It creates a pleasant moonlight effect, great for walking around at night after you've been sleeping, since it's barely any light - but it's a cool-ish white (like the moon). For some of my fixtures that have bulbs without diffusers, you can even get crisp shadows that look just like moonlight shadows. I love it.

My point is - the functionality in the PR seems like it kind of assumes the sleep color temp is lower (on the kelvin scale) than the min color temp, and that's not how all users use the sleep settings.

Good idea! I stumbled on this effect when about a year ago I replaced my bulbs in the bathrooms with 6500K bulbs. I was worried about how harsh that color temp would appear at night. I reset the (non-smart) dimmers to dim very low. I noticed at night when I turned on the light in the bathroom to the minimum brightness, it gave a very pleasant moonlight effect.
I am now embarking on changing as many lights as I can with DIY continuously variable CCT. At first I thought ramp down to low temps at night, but then I remembered the moonlight effect.
Not sure what the right approach would be. With the current functionality, you don't get the ramp down (or up) to the sleep mode temp, but sleep mode itself seems to be sufficient if we're not trying to ramp to it's temp. That's sort of how I would prefer it for the moonlight effect when getting up in the middle of the night.

@rsnodgrass
Copy link

Any update on when this might be merged and released? Thanks!

@th3w1zard1 th3w1zard1 linked an issue Mar 30, 2023 that may be closed by this pull request
@th3w1zard1
Copy link
Collaborator

@basnijholt Hey this PR automerged. It was supposed to wait for all reviewers to approve it and the tests to pass. I'm so sorry.

It's absolutely done though, the config option is the only thing that activates it and it defaults to false.

@basnijholt
Copy link
Owner

I already wondered how this suddenly got in. Thanks for explaining and thanks for getting this over the finish line!

@basnijholt
Copy link
Owner

@all-contributors please add @igiannakas for code

@allcontributors
Copy link
Contributor

@basnijholt

I've put up a pull request to add @igiannakas! 🎉

@basnijholt
Copy link
Owner

@igiannakas
Copy link
Contributor Author

Thank you for the mention 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Ability to set adaptive 'ramp' up and down times
7 participants