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

J1850G compset needs to use CRUv7 data #168

Closed
ekluzek opened this issue Aug 24, 2020 · 1 comment · Fixed by #169
Closed

J1850G compset needs to use CRUv7 data #168

ekluzek opened this issue Aug 24, 2020 · 1 comment · Fixed by #169
Assignees
@ekluzek
Labels
bug
Milestone
CESM2.2

Comments

@ekluzek
Copy link
Contributor

ekluzek commented Aug 24, 2020

Brief summary of bug

@fischer-ncar found the test SMS_Ld5.f09_g17_gl4.J1850G.cheyenne_gnu.allactive-cism-test_coupling fails, because it's using CRUNDEP data version 4, rather than the new version 7 data. We removed the v4 data a while ago on cheyenne to save space. So the compset and the tests should user v7 data now.

General bug information

CESM version you are using: cesm2_2_alpha06e

Machine you are using: cheyenne

Have you modified the code? (If so, please point us to you changes.): No

Details of bug

config_compsets.xml just needs to be updated to use the v7 data.
@ekluzek ekluzek added the bug label Aug 24, 2020
@ekluzek ekluzek added this to the CESM2.2 milestone Aug 24, 2020
@ekluzek ekluzek self-assigned this Aug 24, 2020
@ekluzek
Copy link
Contributor Author

ekluzek commented Aug 24, 2020 •
edited
Loading

Here's the diff I propose...

diff --git a/cime_config/config_compsets.xml b/cime_config/config_compsets.xml
index bdd043d..14ba1bd 100644
--- a/cime_config/config_compsets.xml
+++ b/cime_config/config_compsets.xml
@@ -153,7 +153,7 @@
 
   <compset>
      <alias>J1850G</alias>
-     <lname>1850_DATM%CRU_CLM50%BGC-CROP_CICE_POP2_MOSART_CISM2%EVOLVE_SWAV</lname>
+     <lname>1850_DATM%CRUv7_CLM50%BGC-CROP_CICE_POP2_MOSART_CISM2%EVOLVE_SWAV</lname>
   </compset>
 
   <entries>

ekluzek added a commit to ekluzek/CESM that referenced this issue Aug 24, 2020
@ekluzek
Fix for JG compset fixing ESCOMP#168
d58c538
@ekluzek ekluzek mentioned this issue Aug 24, 2020
Merged
fischer-ncar added a commit that referenced this issue Aug 25, 2020
@fischer-ncar
Merge pull request #169 from ekluzek/fixJG
d9aa797 
Description of changes

Have J1850G compset use CRUv7 data rather than the older version 4 data. We removed the version 4 off local disk to save space on cheyenne and CGD.
Specific notes

Contributors other than yourself, if any: none

Fixes: #168

User interface changes?: No

Testing performed (automated tests and/or manual tests):

SMS_Ld5.f09_g17_gl4.J1850G.cheyenne_gnu.allactive-cism-test_coupling PASSes
@fischer-ncar fischer-ncar mentioned this issue Sep 2, 2020
Closed
RafaPedroFernandez pushed a commit to RafaPedroFernandez/CESM that referenced this issue May 31, 2024
@brian-eaton
Merge pull request ESCOMP#168 from brian-eaton/simple_fix
366ae83 
cam6_2_037?: Changes for Simple Models from Christiane
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ekluzek ekluzek

Labels
bug
Projects
None yet
Milestone
CESM2.2
Development

Successfully merging a pull request may close this issue.

Fix for JG compset fixing #168 ekluzek/CESM
1 participant
@ekluzek