-
Notifications
You must be signed in to change notification settings - Fork 66
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
Harmonizing grids info across resolutions in /g/data/ik11/grids/ #212
Comments
@rmholmes can you assign me to this? Or give me the rights to do so? |
These are consistent
but Also most of the variables in |
OK. So the thing is the derived fields ( |
For the type (Float vs Double), I suppose that means we need to regenerate them by running the model? |
Ah yes, dxt, dyt, dxu, dyu are masked in those files. I'm not sure if that will cause problems with xgcm, but in any case I think it would be best for all of these to be unmasked to match the other grid info ( |
Also, the 0.1 degree grid has |
I'm making the 1 deg data here: /home/156/aek156/payu/make_grids/1deg_jra55_iaf_grid |
see how you go with these instructions: https://github.com/COSIMA/access-om2/wiki/Technical-documentation#outputting-grid-files-with-no-processor-land-masking |
Thanks @aekiss ! |
interestingly, dxt, dyt, dxu, dyu are still masked - maybe that's unavoidable? see |
??! |
see related issues: |
There are different versions of the topography, and some of the old ones are still in use, so we should either provide different versions of the grid files, or omit these fields to avoid confusion
|
There are also different versions of the land mask currently in use, so we should remove the mask from dxt, dyt, dxu, dyu if possible. Note that this is a land mask, not a processor mask. |
|
These are standard output fields so are available via the cookbook already. As they are model-dependent and cannot be unmasked they should be omitted from these unmasked grid files
|
One comment about kmt, kmu files that are perhaps more relevant when moving to CICE6 or versions of CICE5 that need real topography read in is that we are starting to use real integers again in these fields not a 0 1 land se mask, have to admit that it was myself and @DaveBi who made the switch away from ncar's approach in using kmt topography when we moved away from using the mom2.2 approach to setting up ocean topography in grid files. In CICE6 there is a land fast ice scheme that in the Arctic can is impacted by basal ice stress in shallow water on shelves and in straits. In Antarctica fast ice position is driven more by icebergs, ice shelves, ice tounges and coasts. |
Thanks for clarifying this. I'll try to keep it simple and won't add them to the file. |
@rbeucher I tracked down the reason dxt, dyt, dxu, dyu are masked: mom-ocean/MOM5#375 Looks easy to fix. |
Thanks @aekiss ! |
|
These changes in mom remove the masking from dxt, dyt, dxu, dyu and have been compiled into This worked fine to create unmasked grid files at 1 deg and 0.25 deg
but hung every time I tried it at 0.1deg
so I'm currently writing Fortran code to do it, based on @russfiedler's https://github.com/russfiedler/grid_spec_tools |
Thanks for the update and for helping with this @aekiss |
Related issue: automatically discovering grids in ACCESS-NRI catalogue |
This is related to the discussion in #150 .
Issue is that the 0.10, 0.25 and 0.1 grids do not have the same information recorded which make it difficult to visualize outputs using the same notebook recipe. Plotting also relies on the grids being present in
/g/data/ik11/grids
...We need to harmonize the grids and / or investigate the possibility to drop the dependence on those grids.
Could plotting be done using the field's coordinate information? from what I understand the information is not retained in the outputs and we may not want to do so.
I see 2 options:
@navidcy @AndyHoggANU @aidanheerdegen
Also related:
COSIMA/cosima-cookbook#191
The text was updated successfully, but these errors were encountered: