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

Meeting times are off due to daylight savings #347

Open
rossbar opened this issue Nov 8, 2022 · 7 comments
Open

Meeting times are off due to daylight savings #347

rossbar opened this issue Nov 8, 2022 · 7 comments
Labels
question Further information is requested

Comments

@rossbar
Copy link
Member

rossbar commented Nov 8, 2022

Please describe the bug or issue including how to reproduce it.

I'm not sure exactly which meetings are affected, but at least the NumPy community + triage meetings and the NetworkX community meeting are now 1 hr off after the time change. This is likely due to the fact that the meetings times are specified in UTC (+00:00 offset). I'm not sure what the appropriate syntax is for properly handling timezones.

Add any other context about the problem here, screenshots, etc.

No response

@rossbar rossbar added the bug Something isn't working label Nov 8, 2022
@stefanv
Copy link
Member

stefanv commented Nov 8, 2022

Depends on whether you consider that a bug or a feature. It's correctly keeping the time according to UTC. If you want to provide it in Californian time, you have to set timezone to US/Pacific.

@rossbar
Copy link
Member Author

rossbar commented Nov 8, 2022

IIRC there was an external (non-tz-related) reason why we specified them in UTC... maybe the calendar widget wasn't working with timezones? I'm not exactly sure, but I expect that the meeting times for some communities may need to be reviewed to ensure they're correct.

@rossbar rossbar added question Further information is requested and removed bug Something isn't working labels Nov 8, 2022
@tacaswell
Copy link
Contributor

The calendar widget needs some sort of indicator (or selector!) for what timezone the times shown are in. It is easy to assume that it is "smart" enough to sort out the readers local timezone and grow confused .....

@tacaswell
Copy link
Contributor

Actually, I take that back it looks like it should be picking up the "local" timezone, but at least for Matplotlib something is wrong with how the timezone is being read...

@stefanv
Copy link
Member

stefanv commented Feb 28, 2023

@tacaswell Is the widget rendering your meetings differently from, say, Google Calendar then?

@QuLogic
Copy link

QuLogic commented Mar 2, 2023

For some clarity, here is the monthly view:
image
There appear to be some times on there, but they are somewhat cut off.

And here is the list:
image

Matplotlib is at 9:00pm on that list. Assuming that's UTC (because it's definitely nowhere near local time for me), that's 16:00EST for me. In Google calendar, it appears at 15:00EST:
image

@kolibril13
Copy link

I also find the calendar quite confusing.
Maybe there can be a note saying what time zone is used?
E.g. a note stating "all times are in the time zone from your current computer time zone setting" (if that's the actual case.)

Also, I've noted that the Matplotlib community call times are one hour off:
image

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

No branches or pull requests

5 participants