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

1deg jra55do ryf.cherrypicks #44

Merged
merged 7 commits into from
Aug 23, 2024

Conversation

anton-seaice
Copy link

Companion PR to ACCESS-NRI/access-om3-configs#91

The input folder was been renamed to INPUT, and now WW3 can't find the restart file :

Model definition file read.

 *** WAVEWATCH III WARNING IN W3IORS : 
     NO READABLE RESTART FILE, INITIALIZE WITH 'CALM' INSTEAD
     IOSTAT =   29

 Restart file read; cold start (calm).

Suggestions @ezhilsabareesh8 ?

@dougiesquire
Copy link
Collaborator

dougiesquire commented Aug 21, 2024

@anton-seaice can you please remove the first commit (db244fa)? We haven't tested this yet so I don't think we want to be propagating it across our configs - see my comment in the original PR.

In fact, I think we should revert this commit from MOM6-CICE6/1deg_jra55do_ryf until the appropriate testing is done - see this comment.

UPDATE: I've submitted a PR to revert the commit from MOM6-CICE6/1deg_jra55do_ryf.

@ezhilsabareesh8
Copy link

@anton-seaice The input directory name in wav_in has to be changed.

initfile = "./input/restart.ww3"

dougiesquire and others added 4 commits August 23, 2024 09:15
Flag OCN_modelio::pio_* params as unused in nuopc.runconfig

remove extra lines in nuopc.runconfig

update comment for ocean nc type
- work-around symlink restart bug in CICE
- concatenate CICE daily output
- auto-generate an intake-esm datastore
…er at this point, and the userscript to concat ice daily output expects the default name to be used. See COSIMA/access-om3#201
@anton-seaice anton-seaice marked this pull request as ready for review August 23, 2024 00:27
@anton-seaice anton-seaice requested review from dougiesquire and ezhilsabareesh8 and removed request for dougiesquire August 23, 2024 00:27
@anton-seaice
Copy link
Author

This is ready to go.

nuopc.runconfig Outdated Show resolved Hide resolved
* Add comments after ocn_cpl_dt in nuopc.runconfig - COSIMA/access-om3#157 (#95)

* Update nuopc.runconfig

Co-authored-by: Andrew Kiss <[email protected]>

Fix bug in nuopc.runconfig by removing the equal sign ('=') from comments (after '#') (#98) (#99)

Co-authored-by: minghang.li <[email protected]>
Copy link
Collaborator

@dougiesquire dougiesquire left a comment

Choose a reason for hiding this comment

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

Thanks @anton-seaice. LGTM and seems to run successfully as is.

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.

4 participants