ESCOMP / ESCOMP/CTSM

build-namelist improvement, automatically add $nl_flags to add_defaults calls

Open
#3,504 0 comments 0 reactions 0 assignees View on GitHub
b4b code health enhancement size: small
Dominant language
Fortran
Stars
352
Forks
361
Avg merge
6d 6h
Merged PRs (30d)
8

Description

It's easy to forget to add needed attributes to both the namelist_defaults XML file as well as the perl code. This happens to all of us, and we struggle with why the namelist isn't working as expected until we remember this key detail.

If we make $nl_flags a required argument to add_default calls, ALL of the flags in $nl_flags could be automatically used as attributes. Which means it would be unlikely you'd need to update the perl code with many changes to namelist options.

This would be a time savings for development.

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.