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

AEST Time Is Incorrect For DayLight Savings In October #2820

Closed
1 task done
greensleeves149 opened this issue Oct 9, 2022 · 3 comments
Closed
1 task done

AEST Time Is Incorrect For DayLight Savings In October #2820

greensleeves149 opened this issue Oct 9, 2022 · 3 comments
Labels

Comments

@greensleeves149
Copy link

What happened?

On the first Sunday in October each year, the AEST time zone moves forward by one hour. Last Sunday (2nd October), the current time displayed in the WLED app was NOT updated to reflect the new Daylight savings time. (I saw this under Time and Macros>Time Setup>Current local Time.) I ended up requesting help in your Discord server and I had "blazoncek" kindly investigate and he came back to me with what we think is the reason why it didn't update on the 2nd October.

In the attached file, I think the issue is that the first line of code with
tcrDaylight = {Second, Sun, Oct, 2, 660} //AEDT = UTC + 11 Hours

I'm guessing that the "Second" refers to the Second Sunday of October to start Day Light Savings Time?
Right now as I type this, it's Sunday 9th October (Second Sunday) and I've noticed that the "Current Local Time" is now CORRECT. However, for the last 7 days, it has been incorrect, meaning all my lights turn on and off at the Non Day Light Saving Time.

So maybe the fix is:
tcrDaylight = {First, Sun, Oct, 2, 660} //AEDT = UTC + 11 Hours

AEST Daylight Savings Issue

To Reproduce Bug

'Current Local Time' in the WLED app in Australia between the first Sunday of October to the Second Sunday of October is incorrect as it fails to advance the time forward by one hour to reflect the new Day Light Savings time.

Expected Behavior

On the first Sunday in October each year, AEST time needs to advance forward by one hour. For example if the old time was 10AM, the WLED Current Local Time should show 11AM. (Daylight savings time ends in the first Sunday of April- which I think is correct in the code above. )

Install Method

Self-Compiled

What version of WLED?

WLED 0.13.3

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

No response

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@domdfcoding
Copy link

I think it's worth double checking the other timezones too. For instance NZ is down as second Sunday in September, but according to the New Zealand government it changes on the last Sunday.

https://github.com/Aircoookie/WLED/blob/e362b3b6aa524441c1ab963c4a470a207e83f922/wled00/ntp.cpp#L100-L104

@Aircoookie
Copy link
Member

Thank you for bringing this to my attention! Both NZ and AEST should now be fixed in 0_14 :)

@greensleeves149
Copy link
Author

Great. Thank you for actioning this so quickly too.

softhack007 added a commit to atuline/WLED that referenced this issue Oct 20, 2022
- issue wled#2820 AEST Time Is Incorrect
- PR wled#2840 Add PKT time zone
- issue wled#2708 4LD unnecessary use of software I2C on ESP32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants