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

ice and waves need same pes #172

Closed
DeniseWorthen opened this issue Aug 31, 2020 · 5 comments
Closed

ice and waves need same pes #172

DeniseWorthen opened this issue Aug 31, 2020 · 5 comments
Labels
enhancement New feature or request

Comments

@DeniseWorthen
Copy link
Collaborator

DeniseWorthen commented Aug 31, 2020

Description
With the current ESMF version used in UFS-S2S, ice and waves need to run on the same PEs.

Solution
The default_vars.sh need to be changed to have ICE and WAV run on same PEs.

Alternatives
An update to ESMF has been tested that will allow us to run on the different PEs in the future.

Additional context
PR #160

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Aug 31, 2020 via email

@DeniseWorthen
Copy link
Collaborator Author

I wrote it incorrectly. I've fixed the issue description. Current ESMF we need same PEs. An update (which Rocky had me test) allows us to run on different PEs.

@rsdunlapiv
Copy link
Collaborator

rsdunlapiv commented Sep 1, 2020

ESMF_8_1_0_beta_snapshot_27 contains the fix for this. It should allow running WAVE and CICE on separate PETs.
https://github.com/esmf-org/esmf/tree/ESMF_8_1_0_beta_snapshot_27

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Sep 2, 2020 via email

@rsdunlapiv
Copy link
Collaborator

@junwang-noaa yes it should be ready to go

DeniseWorthen added a commit that referenced this issue Sep 8, 2020
* Replaces CICE5 with CICE6: Issue #159 

* Adds required  CICE6 items to nems.configure. Removes non-cmeps nems.configure files. Updates default_vars and edit_inputs to allow CICE6 resolution and processor count to be specified at run time. Updates ice_in_template for CICE6

* Updates to esmf8.1.0bs27 to allow ICE and WAV on separate PEs (Issue #172)

* Adds cpl_mode (Issue #138) and use_coldstart as new configurable variables (Issues #86, #53). 

* Fixes coldstart run sequence by calling the restart write phase before the accum avg phase (Issue #167)
ShanSunNOAA pushed a commit to ShanSunNOAA/ufs-s2s-model that referenced this issue Oct 28, 2020
Top level CMakeLists.txt and CMakeLists.txt file in FV3 have been changed (and cleaned up) to use imported targets (mostly for NCEPLIBS).
Remove *.appBuilder compsets and log.
Remove outdated doc directory
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants