Repository navigation
Conversation
- turn FATAL error when ESMF mask and MOM6 mask differ intol WARNING, so that we can set up a MOM6 domain including ice shelves but avoid ESMF and the mediator transferrring atmospheric data into the ice shelves - use ocean_state%fluxes%frac_shelf_h when computing the So_omask for the mediator from MOM6 data, so that the coupler does not get sent fields in regions it does not expect. A ceiling is used for frac_shelf_h (fraction of ice shelf values) - partial ice shelf coverage may need to be better thought out. - for above to work, the ocean_state_type is made public. - Also initialise ice shelf fluxes and forces separately. Otherwise, shelf_sfc_mass_flux field is initialised which should only be used with dynamic ice shelf (and results in crash when it can't find the file if you're running from a restart.)
Avoids segmentation fault ACCESS-NRI/access-om3-configs#863
This parameter scales the friction velocity computed due to prescribed tides either through UTIDE = constant value or through TIDEAMP = True and associated TIDEAMP_FILE. Jourdain et al. 2019 suggest a value of 0.66 to estimate tidal ice shelf boundary layer velocity from total barotropic tide amplitude. Default is set to 1 so should not change answers
Parameters READ_TIDEAMP and UTIDE not only affect INT_TIDE_DISSIPATION but also ice shelf melt rates via the ice shelf basal melt rate friction velocity calculation. In this commit the effect on ice shelves is added to the description of the parameters to make this clearer.
angus-g
marked this pull request as draft
October 25, 2026 22:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a minimal set of cherry-picked commits from #38, excluding a bespoke flux override and commits that will already come in through GFDL.
ocean_state_typemust be public (https://github.com/ACCESS-NRI/ACCESS-OM3/actions/runs/40137539215/job/121002625095#step:13:2376)dev/gfdl#38 also had a modification to support (dynamic) shelf restarts with NUOPC. When #110 lands, this is no longer required for static shelves since they have no restartable fields. In general, this will need to be addressed for dynamic shelves with NUOPC.