-
Notifications
You must be signed in to change notification settings - Fork 1
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
Don't skip files based on filename #1
Comments
This comparison between all the full-year .nc files
shows that this bug has only affected the following files in
with the first differences occurring at 2023-12-10T15:00:00. So we got lucky, and only Dec 2023 was affected, not the previous years. Even better, 😅 |
The data in |
@aekiss I just reran my Compare This is surprising to me. Did I do something wrong maybe? |
Hm, I'm seeing differences, eg between these
and more |
|
Ah sorry, yes you're right. I had my ncview limits wrong. All good. I'll replace the run in This is the only run I know of which uses |
I've renamed |
This skip based on filename is incorrect, because the upstream repository doesn't update the filename of complete year files, even though it continues to update the data in the file as the finalised reanalysis comes in. So if the script is run in January it will download unfinalised data for December of the previous year, and later re-runs of the script won't update the file because the filename is unchanged and therefore skipped.
JRA55-do-1-5-0-1/update.py
Lines 25 to 26 in 9452f99
The text was updated successfully, but these errors were encountered: