ESCOMP / ESCOMP/CTSM

Deprecate support/testing for NWP with Bgc/BgcCrop

Open
#2,579 2 comments 0 reactions 1 assignee Claimed by @ekluzek View on GitHub
b4b code health good first issue
Dominant language
Fortran
Stars
352
Forks
361
Avg merge
6d 6h
Merged PRs (30d)
8

Description

The main usage for NWP in CTSM is with SP rather than BGC or BGC-Crop. Supporting both NWP with SP and BGC/BGC-Crop adds more options and complexity to our compsets and compset options as well as to our testing. Since, right now we don't envision there is a user base behind NWP with Bgc, we will deprecate it and start removing it as something that can be easily done and tested. Users could still run this way if desired by setting XML options, but it won't be available at the compset level, won't be tested, and users would be on their own if they find problems.

Definition of done:
- [x] Mark it as deprecated in appropriate README files
- [x] Mark it as deprecated in doc/IMPORTANT_NOTES
- [x] Remove any NWP-Bgc tests
- [x] Remove the option from config_compsets.xml
- [x] Remove the capability in config_component.xml (in description)

What we could do:
- [ ] Mark it as a warning in build namelist (can be overridden by adding --ignore_warnings to CLM_BLDNML_OPTS)
- [ ] Above, but as a fatal error

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.