-
Notifications
You must be signed in to change notification settings - Fork 312
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
Transition FATES LUH timeseries test from fixed to transient compset #2304
Comments
This relates to #1617 |
This also makes me wonder if FATES transient cases with LUH can be run with transient lake, or urban? |
Thanks for catching this @ekluzek. Not being very well versed in the compset aliases, is the idea that adding the |
@glemieux yes, by setting HIST_DATM all the things will be set to run as historical transient. So prescribed-aerosols, and urban, and population density and all those things will be transient. Actually, I think what we should do is to remove the LUH testmod and set up the HIST FATES compset to have LUH turned on by default. So LUH will be on anytime a HIST compset is used. Possibly we should also not allow LUH to be on for a non HIST compset. It might be good for us to chat about this at some point? |
Thanks @ekluzek. As you mentioned in the ctsm software meeting today, chatting about this after the 2-stream testing meeting next week sounds like a good plan. |
@glemieux this is what I think this needs to look like. The transient use-cases turn on flanduse_timeseries files as follows: <clm_demand >flanduse_timeseries</clm_demand> In order to get dynamic lake and urban I think that we leave that as is. <fates_harvest_mode use_fates=".true.">luhdata_mass</fates_harvest_mod>
<use_fates_luh use_fates=".true.">.true.</use_fates_luh> Note, I'm assuming the default would be luhdata_mass would be the default, but it can be whatever is desired here. The transient use case will then set urbantv on, and the HIST compset will set DATM forcing to transient as appropriate. ndep and popdens shouldn't matter for FATES. If they do in the future we can add handling for them. |
Some notes about the different ways of handling complex options is here: https://docs.google.com/presentation/d/1FIaQmgN35v3tQxd9hpxcUQZ2CJXFpfx991rZgXwMKzA Use-cases are one of those options and talked about there. |
The LUH test added to ctsm5.1.dev160 ERS_D_Ld30.f45_f45_mg37.I2000Clm50FatesCruRsGs.derecho_intel.clm-FatesColdLUH2, is for a fixed compset rather than a transient compset. This should be changed into a transient compset, with do_transient_pft=FALSE. The LUH timeseries by definition is a transient dataset and should be run is a transient case.
In testing this I found that we need to add a transient compset for FATES. I tried using a long compset name, but got a fail in the build: SMS.f45_f45_mg37.HIST_DATM%GSWP3v1_CLM51%FATES_SICE_SOCN_MOSART_SGLC_SWAV.derecho_intel.clm-FatesColdLUH2
The text was updated successfully, but these errors were encountered: