Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
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

Get FATES working for NEON #1613

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

Get FATES working for NEON #1613

wants to merge 6 commits into from

Conversation

ekluzek
Copy link
Contributor

@ekluzek ekluzek commented Jan 25, 2022

Description of changes

Proposal for changes to get FATES working for NEON

Specific notes

Contributors other than yourself, if any: Whole team in discussing this

CTSM Issues Fixed (include github issue #):
Fixes #1609

Are answers expected to change (and if so in what way)? No

Any User Interface Changes (namelist or namelist defaults changes)?
Added new use-cases: 2018-PD_NEON_transient, PD_control, 2018_control

Testing performed, if any:
So far just the build-namelist testing

@ekluzek ekluzek added type: enhancement new capability or improved behavior of existing capability tag: enh - new science enhancement that brings in new science capabilities labels Jan 25, 2022
@ekluzek ekluzek self-assigned this Jan 25, 2022
@ekluzek
Copy link
Contributor Author

ekluzek commented Jan 25, 2022

At the NEON meeting @adrifoster pointed out that we'll likely need a different list of output history fields for FATES than BGC (and possibly SP as well). I could do that in the user-mod directory, with bash logic so that we don't have to create more user-mod directories. The other way would be to possibly put it in the use case, or more embedded into build-namelist (probably namelist_defaults_ctsm.xml).

@ekluzek ekluzek marked this pull request as draft January 25, 2022 20:25
@ekluzek ekluzek added the PR status: work in progress PR: author feels this is NOT ready to merge to master label Jan 25, 2022
@wwieder
Copy link
Contributor

wwieder commented Jan 25, 2022

yep, good idea here Erik we're using user_mod_dirs to set this up for the NEON case, but a more general way of defining the default history file output for different configurations (SP/BGC/FATES) would be helpful.

@ekluzek ekluzek mentioned this pull request Jan 11, 2023
@wwieder
Copy link
Contributor

wwieder commented Feb 22, 2023

@ekluzek and @adrifoster we still may need some documentation, but is this issue resolved?

@adrifoster
Copy link
Contributor

@ekluzek and @adrifoster we still may need some documentation, but is this issue resolved?

I guess it depends on our definition of done.... we still don't have anything for the crop sites, which is maybe fine. I am working on a solution to update the surface datasets... and run_neon doesn't work for FATES.... but you can run FATES at most of the NEON sites.

@ekluzek ekluzek removed the PR status: work in progress PR: author feels this is NOT ready to merge to master label Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag: enh - new science enhancement that brings in new science capabilities type: enhancement new capability or improved behavior of existing capability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need to get FATES working for NEON
3 participants