Skip to content

#981 Add missing ocean and seaIce grid labels#990

Open
Jared Drayton (mo-jareddrayton) wants to merge 3 commits into
mainfrom
981-add-missing-seaiceocean-variable-grid-labels
Open

#981 Add missing ocean and seaIce grid labels#990
Jared Drayton (mo-jareddrayton) wants to merge 3 commits into
mainfrom
981-add-missing-seaiceocean-variable-grid-labels

Conversation

@mo-jareddrayton

Copy link
Copy Markdown
Collaborator

Closes #981.

A chunk of seaIce and ocean grid labels were missing, partially due to the base defaults still getting picked up. This PR should add labels for all mappings that are present for ukcm2 at this point in the repository.

Some of the ancils I needed to specify manually but not sure if these are the right mappings Matthew Mizielinski (@matthew-mizielinski) ?

    "areacello_ti-u-hxy-u": "tripolar-native",
    "basin_ti-u-hxy-u": "tripolar-native",
    "deptho_ti-u-hxy-sea": "tripolar-native",
    "dxto_ti-u-hxy-u": "tripolar-native",
    "dxuo_ti-u-hxy-u": "tripolar-ugrid",
    "dxvo_ti-u-hxy-u": "tripolar-vgrid",
    "dyto_ti-u-hxy-u": "tripolar-native",
    "dyuo_ti-u-hxy-u": "tripolar-ugrid",
    "dyvo_ti-u-hxy-u": "tripolar-vgrid",
    "hfgeou_ti-u-hxy-sea": "tripolar-native",
    "hfsnthermds_tavg-ol-hxy-sea": "tripolar-native",
    "rsdo_tavg-ol-hxy-sea": "tripolar-native",
    "sftof_ti-u-hxy-u": "tripolar-native",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small tweak to do, but otherwise looks good

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have found a few very minor mis-mappings of variables to grids in the seaice -- all minor modifications

siforceintstrx_tavg-u-hxy-si = ocean tripolar-native
siforceintstry_tavg-u-hxy-si = ocean tripolar-native
siforcetiltx_tavg-u-hxy-si = ocean tripolar-native
siforcetilty_tavg-u-hxy-si = ocean tripolar-native

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these should be tripolar-ugrid (x) or tripolar-vgrid (y) depending on the last letter of the variable name

there will be a variable called siforcecorioly_tavg-u-hxy-si appearing shortly and this should be tripolar-ugrid

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I was doing this based on the dimension names. I guess that isn't reliable for x/y components then?

        float siforceintstrx(time_counter, y_grid_T, x_grid_T) ;
                siforceintstrx:standard_name = "internal_stress_in_sea_ice_x" ;
                siforceintstrx:long_name = "(Internal stress term in force balance (x-component)) * (sea ice area fraction)" ;
                siforceintstrx:units = "N/m2" ;
                siforceintstrx:online_operation = "average" ;
                siforceintstrx:interval_operation = "1 month" ;
                siforceintstrx:interval_write = "1 month" ;
                siforceintstrx:cell_methods = "time: mean" ;
                siforceintstrx:cell_measures = "area: area_grid_T" ;
                siforceintstrx:_FillValue = 1.e+20f ;
                siforceintstrx:missing_value = 1.e+20f ;
                siforceintstrx:coordinates = "time_centered nav_lat_grid_T nav_lon_grid_T" ;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a mistake in the si3 configuration that wasn't discovered until late

Comment on lines +633 to +634
siu_tavg-u-hxy-si = ocean tripolar-native
siv_tavg-u-hxy-si = ocean tripolar-native

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should be tripolar-ugrid and tripolar-vgrid

Comment on lines +625 to +628
sistrxdtop_tavg-u-hxy-si = ocean tripolar-native
sistrxubot_tavg-u-hxy-si = ocean tripolar-native
sistrydtop_tavg-u-hxy-si = ocean tripolar-native
sistryubot_tavg-u-hxy-si = ocean tripolar-native

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've just found these four in the mappings with the coordinate replacement processor

sistrx(dtop|ubot) -> tripolar-ugrid
sistry(dtop|ubot) -> tripolar-vgrid

@mo-jareddrayton Jared Drayton (mo-jareddrayton) force-pushed the 981-add-missing-seaiceocean-variable-grid-labels branch from 858dd9c to e829c34 Compare July 14, 2026 12:30

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One query otherwise good


[ocnBgchem]
fgdms_tavg-u-hxy-sea = atmos latlon-native
fgdms_tavg-u-hxy-sea = ocean latlon-native

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you double check this one -- I think it might need to be atmos as before, because it is output by the UM

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

Successfully merging this pull request may close these issues.

Add missing seaIce/ocean variable grid labels

2 participants