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

Add new ECwISC30to60E3r1 mesh #58

Closed

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Aug 15, 2023

See MPAS-Dev/compass#666 for details on the mesh and E3SM initial condition

@xylar xylar requested review from mark-petersen and jonbob August 15, 2023 13:09
Comment on lines 154 to +158
<config_Redi_horizontal_taper>'ramp'</config_Redi_horizontal_taper>
<config_Redi_horizontal_taper ocn_grid="SOwISC12to60E2r4">'RossbyRadius'</config_Redi_horizontal_taper>
<config_Redi_horizontal_taper ocn_grid="ECwISC30to60E2r1">'RossbyRadius'</config_Redi_horizontal_taper>
<!-- To do: ramp for WC but RossbyRadius for Cryo -->
<config_Redi_horizontal_taper ocn_grid="ECwISC30to60E3r1">'ramp'</config_Redi_horizontal_taper>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@jonbob and @darincomeau, these are the situations that need discussion before this goes into E3SM. We want to use the ECwISC30to60E3r1 (or maybe the IcoswISC30E3r1 if results look better) for both WCYCL and CRYO configurations but there's currently not a way to have different parameter values for the same mesh. How can we handle this?

Comment on lines +294 to +295
logger.warning("WARNING: The specified compset is requesting ocean ICs spunup from a G-case")
logger.warning(" But no file available for this grid.")
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We can add the spun-up initial condition once we have one.

cime_config/config_grids.xml Outdated Show resolved Hide resolved
@xylar xylar requested review from darincomeau and removed request for mark-petersen August 15, 2023 13:41
@xylar xylar force-pushed the mesh/add-ecwisc30to60e3r1 branch from eceb915 to 31a6fdc Compare August 26, 2023 09:44
@xylar xylar changed the base branch from master to alternate August 26, 2023 09:45
@xylar xylar changed the base branch from alternate to master August 26, 2023 09:45
@xylar
Copy link
Collaborator Author

xylar commented Sep 1, 2023

Closing in favor of #63

@xylar xylar closed this Sep 1, 2023
@xylar xylar deleted the mesh/add-ecwisc30to60e3r1 branch September 1, 2023 09:24
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.

1 participant