Skip to content

Commit

Permalink
Merge pull request #189 from mvertens/feature/fix_budget_output
Browse files Browse the repository at this point in the history
### Description of changes
Fixes #188 where budgets are not written out for December or annually for a 1 year run

### Specific notes
The fix is to use nexttime rather than currtime to trigger budget output. This is consistent with the time stamp of history and restart files as well in cmeps.

Contributors other than yourself, if any: None

Fixes: #188 

Are changes expected to change answers?
 - [x] bit for bit
 - [ ] different at roundoff level
 - [ ] more substantial

Any User Interface Changes (namelist or namelist defaults changes)?
 - [ ] Yes
 - [x] No

Testing performed if application target is CESM:(either UFS-S2S or CESM testing is required):
This only effects CESM configurations. 
- Verified that an A compset at f19_g17 run for a year with budgets turned on produced both annual budget information as well as a budget entry for month 12.
- Verified that for a 3 month run, B1850 at f19_g17, budgets were created for month 3 and that the results were bfb with a run without these cmeps changes.

Hashes used for testing:
- [x] CESM:
  - repository to check out: https://github.com/ESCOMP/CESM.git
  - tag: cesm2_3_beta02
  • Loading branch information
jedwards4b authored May 11, 2021
2 parents 13e347a + d227909 commit cf0d47b
Show file tree
Hide file tree
Showing 2 changed files with 133 additions and 122 deletions.
2 changes: 1 addition & 1 deletion mediator/med.F90
Original file line number Diff line number Diff line change
Expand Up @@ -2435,7 +2435,7 @@ subroutine DataInitialize(gcomp, rc)
!---------------------------------------
call med_diag_init(gcomp, rc)
if (ChkErr(rc,__LINE__,u_FILE_u)) return
call med_diag_zero(gcomp, mode='all', rc=rc)
call med_diag_zero('all', rc=rc)
if (ChkErr(rc,__LINE__,u_FILE_u)) return

!---------------------------------------
Expand Down
Loading

0 comments on commit cf0d47b

Please sign in to comment.