Skip to content

Make fates_paramfile appear on input data list - #4176

Open
mvdebolskiy wants to merge 1 commit into
ESCOMP:b4b-devfrom
mvdebolskiy:fix/fates-paramfile-demand
Open

Make fates_paramfile appear on input data list#4176
mvdebolskiy wants to merge 1 commit into
ESCOMP:b4b-devfrom
mvdebolskiy:fix/fates-paramfile-demand

Conversation

@mvdebolskiy

@mvdebolskiy mvdebolskiy commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Description of changes

Fix fates_paramfile not getting on input_data_list and not being checked for existance during case submission.

Specific notes

Moved some calls in main of CLMBuildNamelist.pm and got rid of passing landroot through command line. It is also renamed to lndroot.

Contributors other than yourself, if any:

CTSM issues resolved or otherwise addressed, if any:

  • (Replace this text, including GitHub issue #, and add more list items as needed)

Testing planned or performed, if any:

  • aux_clm on derecho (only namelist comparison with b4bdev)

Checked if fates_paramfile is in ctsm.input_data_list in a I2000Clm50Fates case.

Requirements before merge:

  • The code in this PR branch builds with no errors.
  • The code in this PR branch runs with no errors. N/A
  • This either (a) does not change answers
  • I have reviewed relevant parts of the CLM documentation Tech Note or User's Guide to determine if anything needs to be changed or added. If it does, describe:
  • This PR either (a) does not create a need to update the documentation.

… variable is used instead of passing it around gazillion times.
@mvdebolskiy
mvdebolskiy requested review from ekluzek and rgknox August 14, 2026 13:54
@mvdebolskiy mvdebolskiy added b4b bit-for-bit bug something is working incorrectly labels Aug 14, 2026
@mvdebolskiy

Copy link
Copy Markdown
Contributor Author

The reasoning for removing the cmdl option is that the command is already extreamly crowded with them. Also, land component is refered everywhere as lnd, hence renaming the attribute value.

@ekluzek ekluzek moved this to In progress - master in CTSM: Upcoming tags Aug 24, 2026
@ekluzek ekluzek moved this from In progress - master to In progress - b4b-dev in CTSM: Upcoming tags Aug 24, 2026
@wwieder

wwieder commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Thanks for opening this PR, @mvdebolskiy. This will be a nice enhancement for FATES runs.

@wwieder wwieder added this to the FATES freeze for ctsm6 milestone Aug 24, 2026
@wwieder wwieder assigned wwieder and ekluzek and unassigned wwieder Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

b4b bit-for-bit bug something is working incorrectly

Projects

Status: In progress - b4b-dev

Development

Successfully merging this pull request may close these issues.

3 participants