-
Notifications
You must be signed in to change notification settings - Fork 8
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
make 1deg_jra55do_iaf branch #15
Comments
OK, initial merge done. In merging these I noticed some variables in |
Thanks @aekiss. The offset is required for IAF since some of the variables in JRA55do-IAF start at 0:00 hrs (temperature, velocities,..) and other variables at 1:30 hrs (rainfall flux, snow flux,...), so I set a positive offset of 5400 seconds (1.5hrs) for the variables that starts at 0:00hrs. I tested this configuration and working fine. |
Ah, I didn't realise the offsets are different in the RYF and IAF data. for f in /g/data/ik11/inputs/JRA-55/RYF/v1-4/*.nc; do echo $f; ncdump -v time $f | grep "^ time"; done
for f in /g/data/qv56/replicas/input4MIPs/CMIP6/OMIP/MRI/MRI-JRA55-do-1-4-0/*/*/*/*/*/*1958*.nc; do echo $f; ncdump -v time $f | grep "^ time"; done
The RYF time axes all start at 0 because the RYF data is given a new time axis, effectively applying an offset. IAF data starts at 0.0625 day (5400s) for |
It makes more sense to set a negative offset for variables that start at 5400s and 43200s to align with RYF. I've set an offset of -5400 for |
Great, thanks @ezhilsabareesh8. It would be better to push this to a new branch based on |
Thanks @ezhilsabareesh8 I've just updated the XML generators to match |
@aekiss Looks like this issue has been resolved. Can we close it? |
yep! |
I'll pull together a
1deg_jra55do_iaf
branch, based on1deg_jra55do_ryf
but with the relevant changes from @ezhilsabareesh8's https://github.com/COSIMA/MOM6-CICE6/tree/1deg_jra55_iaf_iss33The text was updated successfully, but these errors were encountered: