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.

Sign up for GitHub

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

Jump to bottom

1deg jra55do iaf.cherrypicks #45

Merged
merged 7 commits into from
Aug 23, 2024
Merged

1deg jra55do iaf.cherrypicks #45

merged 7 commits into from
Aug 23, 2024

Conversation

anton-seaice
Copy link

minghangli-uni and others added 5 commits August 23, 2024 10:38
@minghangli-uni @anton-seaice
Add comments after ocn_cpl_dt in nuopc.runconfig (#97)
3e0056e 
* Add comments after ocn_cpl_dt in nuopc.runconfig - COSIMA/access-om3#157 (#95)

* Update nuopc.runconfig

Co-authored-by: Andrew Kiss <31054815+aekiss@users.noreply.github.com>

---------
@dougiesquire @anton-seaice
Update exe (COSIMA/access-om3#188), use environment module (COSIMA/ac…
550791d 
…cess-om3#168)
@dougiesquire @anton-seaice
Use netcdf4 for MOM6 (COSIMA/access-om3#188)
5efbfee 
Flag OCN_modelio::pio_* params as unused in nuopc.runconfig

remove extra lines in nuopc.runconfig

update comment for ocean nc type
@dougiesquire @anton-seaice
Use Payu userscripts from om3-scripts (COSIMA/access-om3#182) to:
755a747 
- work-around symlink restart bug in CICE
- concatenate CICE daily output
- auto-generate an intake-esm datastore
@dougiesquire @anton-seaice
Update input directory name from "input" to "INPUT" (COSIMA/access-om…
e58e9bb 
…3#170)
dougiesquire
dougiesquire reviewed Aug 23, 2024
View reviewed changes
Copy link
Collaborator

@dougiesquire dougiesquire left a comment •
edited
Loading

Choose a reason for hiding this comment

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

How come the ice_in hist_suffix addition is not included in this one?

Otherwise, LGTM and runs successfully as is.

@anton-seaice
Set hist_suffix for cice history output. This can only be one charact…
9d25aa5 
…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
set cice io consistent with MOM6-CICE6
feb5688
@anton-seaice
Copy link
Author

Thanks! Operator error most likely

dougiesquire
dougiesquire approved these changes Aug 23, 2024
View reviewed changes
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.

Verified again that this runs as is. Thanks @anton-seaice

@anton-seaice anton-seaice merged commit ab8aafd into 1deg_jra55do_iaf Aug 23, 2024
@anton-seaice anton-seaice deleted the 1deg_jra55do_iaf.cherrypicks branch August 23, 2024 04:53
This was referenced Aug 30, 2024
Closed
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dougiesquire dougiesquire dougiesquire approved these changes

@ezhilsabareesh8 ezhilsabareesh8 Awaiting requested review from ezhilsabareesh8

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@anton-seaice @dougiesquire @minghangli-uni