ESCOMP / ESCOMP/CTSM

Rename `*_ICE` diagnostic fields to `*_GLC`

Open
#844 2 comments 0 reactions 0 assignees View on GitHub
b4b enhancement good first issue
Dominant language
Fortran
Stars
352
Forks
361
Avg merge
2d 21h
Merged PRs (30d)
7

Description

Some confusion arose recently connected to the `_ICE` suffix on CTSM history variables. For most history fields, `SOMETHING_ICE` means that this is the SOMETHING field but only over glacier landunits. However, there are a few – such as `QRUNOFF_ICE_TO_COUPLER` – where `ICE` means that this is an ice as opposed to liquid term. So in this case, this is ice runoff averaged over all landunits.

I proposed, and LIWG members supported, that we change the suffix that denotes that a field is only averaged over glacier landunits, using `_GLC` rather than `_ICE` for this purpose.

This will be an easy change that will only affect a handful of history fields (hence I'm giving it the "simple bfb" label). However, it will impact the ability to compare master with CMIP6/ISMIP6 runs, so I'd suggest holding off on this until we're past the point that people will frequently want to do such comparisons with master – e.g., holding off until after water isotopes are in place (hence I'm giving it the "blocked: answer changing" label). (And yes, I recognize that it seems weird to have both of these labels, but both seem appropriate in this case.)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.