ESCOMP / ESCOMP/CAM

remove dyn_time_lvls from all pbuf_add_field calls

Open
#1,532 1 comment 0 reactions 2 assignees Claimed by @jimmielin View on GitHub
Must have CESM3
Dominant language
No language data
Stars
91
Forks
183
Avg merge
6d 2m
Merged PRs (30d)
9

Description

### Issue Type

Code Clean-up

### Issue Description

dyn_time_lvls is for storing multiple time-levels in the pbuf, which was used to accommodate the EUL dycore. The EUL dycore was removed from the code, but there remains a large number of pbuf_add_fields calls that declare a dyn_time_lvls dimension, e.g.,

```
call pbuf_add_field('AST', 'global', dtype_r8, (/pcols,pver,dyn_time_lvls/), ast_idx)
```

this dimension should be removed from all pbuf variables. This simplifies the pbuf_get_field calls to exclude optional arguments ```start=(/1,1,itim_old/), kount=(/pcols,pver,1/)```.

### Will this change answers?

I Don't Know

### Will you be implementing this yourself?

No

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.