-
Notifications
You must be signed in to change notification settings - Fork 184
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
ISO offset vs IANA code ambiguities #376
Comments
whoa, hi Michael, yep this is bad. I'm re-doing this right now, in a branch. It's a bit of a head-scratcher - please allow me:
I may look at how other libraries handle this. I may not have a full understanding of the problem. Maybe unclear inputs like right now you can co-erce the timezone like this: spacetime('2023-03-12T01:39:00-08:00', 'America/Los_Angeles').time()
//1:39am but this should definetly get cleared-up. any help is welcomed. |
goto
around DST produces unexpected results
you know what, I know what's happening. it creates a date at The example i gave works because it knows about the dst change at parse-time, so the correct interpretation is clearer. super bad. not fun. i am working on a re-write to address this. will be the next release, but may be a week or two out. will keep this open, and add it as a test there. |
is this bug fixed and if no , around which file might be causing this bug? |
hey Eshan, sorry - there's no easy fix for this. #235 is going to require a full re-write, in order to fix properly. I plan on working on a fix for it over this year. |
maybe rewriting with having each function having its own file will be easy
to debug
…On Thu, 4 Jan 2024 at 02:15, spencer kelly ***@***.***> wrote:
hey Eshan, sorry - there's no easy fix for this. #235
<#235> is going to
require a full re-write
<#341 (comment)>,
in order to fix properly. I plan on working on a fix for it over this year.
cheers
—
Reply to this email directly, view it on GitHub
<#376 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/APCD3ZUPVHCXF4EOYX566E3YMW7ORAVCNFSM6AAAAAAVXCBVBWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZVHE2DQOBXGE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
We've run into an issue with DST while converting a datetime to a timezone that is already set to that timezone.
Example:
Produces
3:39am
instead of the expected1:39am
.This is the result from running it in the browser:
The text was updated successfully, but these errors were encountered: