Open rgknox opened 4 years ago
Just re-reading this. Sorry for the delay. I think it does indeed make more sense for soil BGC parameters indexed by PFT to be in the FATES file and soil BGC parameters -not- indexed by PFT to be in the HLM file. In the case of thinking about mycorhizal associations, I feel like we might still find ourselves with something of a grey area, but I think that swallowing the soil parameters into FATES would be a less sensible route...
Just to say that I also support only adding those soil BGC parameters that are indexed by PFT. And I think we should probably plan on mycorrhizal parameters being in that set too, once implemented.
There is only one parameter for eca that is plant specific, but not pft specific: fates_eca_plant_escalar. I'm fine with letting the HLM file be its holder.
We currently still tie decomposer biomass (when ECA is on) to the PFT, I was unsure if this was the best thing to do:
Moving forward with the implementation of nutrient acquisition between plants and soil competitors, I'm running into some weird questions about where to keep parameters. Note that the acquisition code is on the host model side (i.e. in ELM).
When using FATES-ECA-ELM, some parameters are inherently plant related, and must be associated with FATES PFTs. It is impossible not to define them in the FATES parameter file.
However, there are some "ECA" parameters that are not plant related, but involved in that same code set, like the vmax and km of the mineral surfaces or decomposers.
In one hand I think it would be nice to have all BGC parameters in 1 spot and define them in our file. On the other hand, users may be just as confused to find soil parameters in the vegetation model, and further we will become beholden to hold all of the soil BGC parameters which I think is a slippery slope and we will start taking on too many parameters.
My preference is to only keep "plant specific parameters" in FATES, and leave all other BGC parameters in the host land-model.
I'm currently working with Xiaojuan and @thorntonpe on getting parameter set for FATES-RD.
Current Set of Plant-Specific FATES-ECA parameters, thanks @qzhu-lbl:
(Some) Non Plant Specific Parameters (example for ECA):