Skip to content
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

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

Open
aekiss opened this issue May 5, 2021 · 9 comments
Open

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

aekiss opened this issue May 5, 2021 · 9 comments

Comments

@aekiss
Copy link
Contributor

aekiss commented May 5, 2021

/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 #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
Copy link
Contributor Author

aekiss commented May 27, 2021

related issue: #233

@hakaseh
Copy link
Collaborator

hakaseh commented Jul 12, 2021

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
Copy link
Contributor Author

aekiss commented Jul 12, 2021

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

@hakaseh
Copy link
Collaborator

hakaseh commented Jul 12, 2021

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
Copy link
Contributor Author

aekiss commented Jul 13, 2021

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
Copy link
Contributor

navidcy commented Oct 11, 2021

Yes let's do this update!

@navidcy
Copy link
Contributor

navidcy commented Oct 11, 2021

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
Copy link
Contributor Author

aekiss commented Jan 25, 2023

We are addressing these problems here: COSIMA/cosima-recipes#212

@aekiss
Copy link
Contributor Author

aekiss commented Jan 25, 2023

I now think ht, hu, kmt and kmu should be deleted rather than updated - see COSIMA/cosima-recipes#212 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants