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

Updated grid aliases in cesm/config_grids.xml #1585

Merged
merged 3 commits into from
May 23, 2017

Conversation

alperaltuntas
Copy link
Member

@alperaltuntas alperaltuntas commented May 22, 2017

Removed grid aliases that do not specify the mask.
Renamed grid aliases so the mask specifier always follow the ocean grid.

Test suite:
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes

User interface changes?:

Code review:

Copy link
Contributor

@jedwards4b jedwards4b left a comment

Choose a reason for hiding this comment

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

Fill in the initial comment template, removing text in [] and replacing with relevant information if any.

It looks like you didn't start from the most recent version of the file config_grids.xml - it is replacing stuff that was recently removed.

@@ -8,11 +8,11 @@
=========================================
The notation for the grid longname is
a%name_l%name_oi%name_r%name_m%mask_g%name_w%name
where
where
Copy link
Contributor

Choose a reason for hiding this comment

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

Please avoid adding unnecessary whitespace to files, it makes the review process difficult. Most editors can be configured to do this.

@@ -1655,6 +1550,17 @@
<map name="ROF2OCN_ICE_RMAPNAME">cpl/cpl6/map_r01_to_gx1v6_120711.nc</map>
</gridmap>

<!--- river flooding variables -->
Copy link
Contributor

Choose a reason for hiding this comment

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

These don't belong in this file and were removed in PR #1535

@jedwards4b jedwards4b merged commit e9a944c into ESMCI:master May 23, 2017
@rljacob rljacob added Responsibility: CESM Responsibility to manage and accomplish this issue is through CESM tp: config labels May 23, 2017
fischer-ncar added a commit that referenced this pull request May 24, 2017
Reverts #1585
We need to back this out until the components are ready to comply.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Responsibility: CESM Responsibility to manage and accomplish this issue is through CESM tp: config
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants