-
Notifications
You must be signed in to change notification settings - Fork 0
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
Time calculation issue #9
Comments
Hello Breeze-Hu,
Read that as offset divided by 24 hours gives the number of days, from which we subtract full years of 365 days, then we subtract leap days for every 4th year, but 15 century years (Y100, 200, 300, 500, etc) were not leaps years so we add those back, yields 2 resulting days. Add 2 to the difference in whole years gives Python I noticed that you found the same discrepancy in some other packages, while others align with the There is currently no easy solution for this. |
Wow! Very admirable! There was exposure to bits and pieces of the difference between the two calendars in the last few days, but not a good understanding or calculation of how the difference between the two days got there. After your explanation I finally understand the reason for this result! One calendar had 15 century years before 1582, again all leap years. But the other has only 400, 800 and 1200 years as leap years. There is a difference of 12 (=15-3) years in between. Thank you very much for your reply! This solved my last two days of confusion. Thank you very much! |
This issue has now been addressed in commit 5dc3d42.
This will be included in upcoming release 1.5.0 of CFtime. Thank you for raising this issue. |
Great! Happy to see this issue resolved, thanks for your work! |
Hello, developers.
I'm using CFtime to do calculations on time and I'm finding inconsistencies with other software and packages (Panoply and python: xarray, cftime,).
R:
Python:
Other more detailed information can be found in this: Offset of “time” dim for terra packages when reading nc file
The text was updated successfully, but these errors were encountered: