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

MOM WIND_STAGGER #258

Closed
anton-seaice opened this issue Jan 20, 2025 · 3 comments
Closed

MOM WIND_STAGGER #258

anton-seaice opened this issue Jan 20, 2025 · 3 comments
Assignees

Comments

@anton-seaice
Copy link
Contributor

It looks like we've missed setting this value in MOM_input:

From the MOM_docs output

WIND_STAGGER = "C"              ! default = "C"
                                ! A case-insensitive character string to indicate the staggering of the input
                                ! wind stress field.  Valid values are 'A', 'B', or 'C'.

This should be A unless i've misunderstood something.

(could be significantly answer changing!)

also N.B. @kieranricardo

p.s. (its set correctly for CICE through the grid_atm line in ice_in)

@dougiesquire
Copy link
Collaborator

The value is hardcoded to A with the CESMCOUPLED directive. So I think we're okay, but I agree this is unclear and we should probably also set it in MOM_input for clarity.

@anton-seaice
Copy link
Contributor Author

Oh good !

@anton-seaice
Copy link
Contributor Author

This has been completed in dev- branches :

1deg_ryf ACCESS-NRI/access-om3-configs#161
1deg_iaf ACCESS-NRI/access-om3-configs#167
1deg_ryf_wombatlite ACCESS-NRI/access-om3-configs#168
025deg_ryf ACCESS-NRI/access-om3-configs#169

And will be captured in #283 and #284

@github-project-automation github-project-automation bot moved this from In Progress to Done in ACCESS-OM3 025 Feb 20, 2025
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

2 participants