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

Unconventional sunrise and sunset times . #105

Open
00schteven opened this issue Mar 17, 2021 · 5 comments · May be fixed by #535
Open

Unconventional sunrise and sunset times . #105

00schteven opened this issue Mar 17, 2021 · 5 comments · May be fixed by #535

Comments

@00schteven
Copy link

I am trying to configure Adaptive lighting for my indoor garden. I would like lights to come on at 16:00 and off at 04:00. Setting manual Sunrise and sunset times in the configuration returns this error:

ValueError: Test: The sun events ('sunset', 'solar_noon', 'sunrise', 'solar_midnight') are not in the expected order. The Adaptive Lighting integration will not work! This might happen if your sunrise/sunset offset is too large or your manually set sunrise/sunset time is past/before noon/midnight.

I think I can set the server location or clock to a time zone more compatible, but it would be nice to maintain a local time zone for the rest of the system.

@RubenKelevra
Copy link
Collaborator

I agree, this should be possible and the verification code for the time input is probably too restrictive

@basnijholt
Copy link
Owner

I never considered this use case. It should indeed be possible to support this case.

@th3w1zard1
Copy link
Collaborator

@00schteven I believe I've implemented this in #535

if someone could test, that'd be great!

@b-rad15
Copy link

b-rad15 commented Nov 2, 2023

Been a while since this lol but I'm testing using this version now and it appears to be working

@b-rad15
Copy link

b-rad15 commented Nov 3, 2023

Rebased #535 in #830

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

Successfully merging a pull request may close this issue.

5 participants