search

COSIMA / access-om2

Deprecated ACCESS-OM2 global ocean - sea ice coupled model code and configurations.
21 stars 23 forks source link

update /g/data/ik11/grids/ocean_grid_*.nc #236

Open aekiss opened 3 years ago

aekiss commented 3 years ago

/g/data/ik11/grids/ocean_grid_*.nc need updating.

/g/data/ik11/grids/ocean_grid_10.nc and /g/data/ik11/grids/ocean_grid_025.nc use the old topography and land mask, so ht, hu, kmt and kmu should be updated to match what's in /g/data/ik11/inputs/access-om2/input_20201102, which includes all the improvements discussed in https://github.com/COSIMA/access-om2/issues/158.

/g/data/ik11/grids/ocean_grid_01.nc currently contains a much smaller set of variables, has variable names that differ from what's used in the coarser ones, and is missing dimension data. It would be much more useful if this was made consistent with the others. I'm not sure whether the variables depth_t and num_levels already match topog.nc and kmt.nc in /g/data/ik11/inputs/access-om2/input_20201102, so that would also be worth checking.

It may also be worthwhile keeping the current versions of /g/data/ik11/grids/ocean_grid_10.nc and /g/data/ik11/grids/ocean_grid_025.nc for those using the old topography.

aekiss commented 3 years ago

related issue: https://github.com/COSIMA/access-om2/issues/233

hakaseh commented 3 years ago

I just saw this issue now. @aekiss where are the grid files with new topography and land mask located? I've been using the old ones for interpolation of BGC initial conditions.

aekiss commented 3 years ago

The config.yaml files in the current configurations use MOM inputs from /g/data/ik11/inputs/access-om2/input_20201102/mom*

hakaseh commented 3 years ago

oh, i was using /g/data/ik11/grids/ocean_grid_10.nc for getting geolon_t and geolat_t, which are not affected by the new topography or land mask. so it's all good actually :)

aekiss commented 3 years ago

I've updated the first post to specify that ht, hu, kmt and kmu should be updated. Purely grid-related variables like geolon_t and geolat_t are OK.

navidcy commented 3 years ago

Yes let's do this update!

navidcy commented 3 years ago

And let's make sure that the grids across all resolutions contain the same variables! Currently they don't and this creates an impasse in comparing results across models with different resolutions.

aekiss commented 1 year ago

We are addressing these problems here: https://github.com/COSIMA/cosima-recipes/issues/212

aekiss commented 1 year ago

I now think ht, hu, kmt and kmu should be deleted rather than updated - see https://github.com/COSIMA/cosima-recipes/issues/212#issuecomment-1403123440