-
Notifications
You must be signed in to change notification settings - Fork 87
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 'transition colortemperature' #59
base: main
Are you sure you want to change the base?
Conversation
Upvoting this to be merged into the main branch. It is quite jarring that the color temperature drops to the minimum when the sun sets - would be great if we could define a sunset color temperature and the minimum color temperature so that there is a smoother transition overnight to that target. |
I will review as soon as I have time. Sorry for not responding sooner. I'm not going to guarantee that this is going to be merged, however. The entire purpose of Circadian Lighting is to maintain your body's natural rhythm - most of the decisions I have made in regards to functionality are based on the scientific research I have studied. Circadian Lighting is not designed to be an aesthetic mechanism and my general position is that I will not change or add functionality simply because of personal preference. I don't want to sound harsh or closed minded but I do need to keep the core mission in mind. I'm not completely ruling out this PR, but I have to take time to research and consider the implications, more than just that the code is functional. I'm open to looking at any scientific papers you can find that suggest that natural color temperature varies after the sun sets, or that implementing such a mechanism is advantageous for maintaining your natural rhythm. |
It sounds like for your use case you'd be best off just configuring You're right though, that this likely wouldn't have negative impacts. So the only real hesitation I have against merging would be that CL is already relatively complicated and requires a lot of description for each option. I really need to find the time to allow configuring CL through the integrations panel so that it can be a more step-by-step process...I digress. Anyway, I will likely merge, just need to get some free time to look it over and do some quick testing. |
I'm now experimenting with some hue sensors to measure light in the house and I do want the lights to turn on during the day (when it get dark) but I don't want them to be on max_brightness. |
I didn't like that the color temperature would switch to 'min_colortemp' when the sun sets or rises. I wanted to have a more subtle transition between day and night.
So I added an optional 'transition_colortemp' setting that determines what the colortemperature should be when the sun sets or rises. If it is not defined, behaviour should not change. This change is backwards-compatible.
My config settings (with Ikea Tradfri lights on Philips Hue):
circadian_lighting:
min_colortemp: 2200
transition_colortemp: 2700
max_colortemp: 4500
So during the day transition is from 4500 to 2700, when the sun sets it goes from 2700 to 2200.