-
Notifications
You must be signed in to change notification settings - Fork 81
Leap seconds #346
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
Comments
I just did a moment's research on this and it frankly sounds like a nightmare. From the Wikipedia page on Leap seconds (see particularly the bolded bit at the end – emphasis mine):
So the best we could do would seem to be a hard-coded list of past times when a leap second was introduced. But even then, I can only begin to envision the mess this would cause. For example, any kind of arithmetic on times would become extraordinarily complex in the presence of leap seconds. (e.g., connected to the other recent NASA issue, modulo on times would much more often need to be left undefined.) So, unless there is a strong need for this, this does seem like something best to just document as unhandled. |
The idea of handling leap seconds came up on the NASA call. They don't need them now, and they don't foresee needing them any time soon. However, they were just wondering if we had thought about them. It would be good to at least document the fact that we don't consider them in the TimeManger. Also, while we are looking at changing some functionality in the TimeManager, it might make sense to think about the issue a bit.
The text was updated successfully, but these errors were encountered: