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

SMS_P720x1_Ln3.hcru_hcru.I2000Clm50BgcCruGs.cheyenne_intel.clm-coldStart should be changed to _Ln6 #874

Closed
billsacks opened this issue Dec 30, 2019 · 0 comments · Fixed by #897
Closed

SMS_P720x1_Ln3.hcru_hcru.I2000Clm50BgcCruGs.cheyenne_intel.clm-coldStart should be changed to _Ln6 #874

billsacks opened this issue Dec 30, 2019 · 0 comments · Fixed by #897
Assignees
@negin513
Labels
bfb bit-for-bit testing additions or changes to tests

Comments

@billsacks
Copy link
Member

As @mvertens points out, SMS_P720x1_Ln3.hcru_hcru.I2000Clm50BgcCruGs.cheyenne_intel.clm-coldStart doesn't run long enough to hit a ROF coupling interval. This is okay with the mct-based cpl7, but fails with cmeps. We should change this to _Ln6.
@billsacks billsacks added tag: simple bfb testing additions or changes to tests labels Dec 30, 2019
negin513 added a commit to negin513/ctsm that referenced this issue Jan 23, 2020
@negin513
change Ln3 to Ln6 in test suites to enable the related test (SMS_P720…
fac4b4b 
…x1_Ln3.hcru_hcru.I2000Clm50BgcCruGs.cheyenne_intel.clm-coldStart) pass for cmpes. (Issue ESCOMP#874).
@negin513 negin513 mentioned this issue Jan 24, 2020
Merged
negin513 added a commit to negin513/ctsm that referenced this issue Jan 29, 2020
@negin513
change Ln3 to Ln6 in test suites to enable the related test (SMS_P720…
6396f73 
…x1_Ln3.hcru_hcru.I2000Clm50BgcCruGs.cheyenne_intel.clm-coldStart) pass for cmpes. (Issue ESCOMP#874).
negin513 added a commit to negin513/ctsm that referenced this issue Feb 6, 2020
@negin513
change Ln3 to Ln6 in test suites to enable the related test (SMS_P720…
a74d8a9 
…x1_Ln3.hcru_hcru.I2000Clm50BgcCruGs.cheyenne_intel.clm-coldStart) pass for cmpes. (Issue ESCOMP#874).
billsacks added a commit that referenced this issue Feb 6, 2020
@billsacks
Merge branch 'bfb_pr_1'
5cc4ede 
Some BFB Fixes: Resolve issues #683, # 874, #878, # 885, # 745, #838

Resolved issues:

#683 :  Writing out FATES parameters only on masterproc.
#874 :  Changing the number of timesteps to run for
                   SMS_P720x1_Ln3.hcru_hcru.I2000Clm50BgcCruGs.cheyenne_intel.clm-coldStart
                   to prevent failing with CMEPS.
#878 :  Removing the unused atm2lnd field.
#885 :  Changing the defaults for br_root in namelist_defaults to
                   enable FUN with CLM4.5.
#745 :  Removing ReadNL private subroutine from initVerticalMod.F90.
#838 :  Clarifying ZISNO in the variable long name.
@samsrabin samsrabin added simple bfb bit-for-bit labels Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@negin513 negin513

Labels
bfb bit-for-bit testing additions or changes to tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Some BFB Fixes: Resolve issues #683, # 874, #878, # 885, # 745, #838. negin513/ctsm
3 participants
@billsacks @samsrabin @negin513