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

memory leak in med_diag_mod.F90 #197

Closed
mvertens opened this issue May 31, 2021 · 1 comment
Closed

memory leak in med_diag_mod.F90 #197

mvertens opened this issue May 31, 2021 · 1 comment
Assignees
@jedwards4b @mvertens
Labels
bug Something isn't working

Comments

@mvertens
Copy link
Collaborator

mvertens commented May 31, 2021 •
edited
Loading

There is a memory leak in med_diag_mod.F90. When running the following test:
SMS_Ly1_Vnuopc.T62_g17.2000_DATM%NYF_SLND_DICE%SSMI_DOCN%DOM_DROF%NYF_SGLC_SWAV_SESP
Out of the box there are no diag calls in the run sequence and there is no memory leak that appears in the med.log file.
However, when the following calls are added to the run sequence

  MED med_phases_diag_atm
  MED med_phases_diag_rof
  MED med_phases_diag_accum
  MED med_phases_diag_print

A memory leak appears in the med.log file.
@mvertens mvertens added the bug Something isn't working label May 31, 2021
@mvertens mvertens mentioned this issue Jun 1, 2021
Merged
3 tasks
jedwards4b added a commit that referenced this issue Jun 1, 2021
@jedwards4b
Merge pull request #195 from mvertens/feature/validation_fixes
e67f3de 
### Description of changes

Bug fixes for CESM 1850 mct/nuopc validation - including memory leak in med_diag_mod.F90.

### Specific notes
These are changes needed to the budgets and run sequence needed that were needed to fix problems in the B1850 mct/nuopc validation.

Contributors other than yourself, if any: @jedwards4b 

CMEPS Issues Fixed: #190 #197 

Are changes expected to change answers?
 - [x] bit for bit

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

Testing performed if application target is CESM:(either UFS-S2S or CESM testing is required):
5 year B1850 1 year simulation was run

Hashes used for testing:
- [x] CESM: 
  - repository to check out: https://github.com/ESCOMP/CESM.git
  - tag: cesm2_3_alpha03a
@jedwards4b
Copy link
Collaborator

fixed in #195

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jedwards4b jedwards4b

@mvertens mvertens

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jedwards4b @mvertens