Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Migrate FATES parameter file switches to the namelist #6918

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

glemieux
Copy link
Contributor

@glemieux glemieux commented Jan 20, 2025

Moves switch options located in the FATES parameter file into the ELM namelist.

Addresses NGEET/fates#1295.

To be coordinated with NGEET/fates#1300.

[non-BFB] for FATES tests only

This commit facilitates moving this switch from the fates parameter file
to the API.  Note that the actual daylength factor value is always
passed through the API.  This just adds the switch to tell FATES to use
it.
@glemieux
Copy link
Contributor Author

To do:

@glemieux glemieux requested a review from rgknox January 20, 2025 19:54
@rljacob rljacob added the FATES label Jan 21, 2025
This commit facilitates moving this switch from the FATES parameter file
to the namelist
This commit facilitates moving this switch from the FATES parameter file
to the namelist
This commit facilitates moving this switch from the FATES parameter file
to the namelist
This commit facilitates moving this switch from the FATES parameter file
to the namelist
This commit facilitates moving this switch from the FATES parameter file
to the namelist
This commit facilitates moving this switch from the FATES parameter file
to the namelist
This commit facilitates moving this switch from the FATES parameter file
to the namelist
This commit facilitates moving this switch from the FATES parameter file
to the namelist
@glemieux glemieux force-pushed the fates-parameter-namelist_switch-migration branch from 019da9d to db8e2cb Compare January 21, 2025 18:56
@rljacob rljacob added the Land label Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants