Remove variables to exclude for fates_sp user mod#4079
Conversation
|
This pull request is optioned per the discussion question in NGEET/fates#1568 (comment) and is being created as a draft as a point of discussion per @wwieder suggestion. Alternatively, this PR could be superseded by adopting build namelist changes that @mvdebolskiy noted that are being integrated into NorESM/CTSM per NorESMhub#224. Since the relates to compset updates cc'ing @ekluzek for insight and input. |
fates_sp user mod
fates_sp user modfates_sp user mod
|
It's definitely a good change to simplify user-mods and put fincl/fexcl lists into the code for major configurations. fates_sp is recognized as a major configuration. So between the buioldnml and the FORTRAN code it should do what's needed to set it up, rather than having to rely on user-mods. It would be good to then be able to remove the user-mods for fates_sp. |
@ekluzek it sounds like you're in favor of moving this PR forward as is, correct? And that removing the fates_sp user mod would be done later? If so, I'll pull this out of draft status. |
Yes absolutely! |
|
Seems like this can go to b4b-dev. @glemieux will check that it doesn't change answers in testing. |
Description of changes
Removes the
hist_fexlfields in thefates_spuser mod. This should be coordinated with NGEET/fates#1568, which adds satellite phenology mode aware logic to automatically set the active/inactive FATES history variables.Specific notes
Contributors other than yourself, if any:
@mvdebolskiy
CTSM issues resolved or otherwise addressed, if any:
If answers are expected to change, describe (delete this line otherwise):
Any user interface changes (namelist or namelist defaults changes)?
Requirements before merge: