CH-Earth / summa

Structure for Unifying Multiple Modeling Alternatives:
http://www.ral.ucar.edu/projects/summa
GNU General Public License v3.0
80 stars 104 forks source link

Rename variables for file manager, update conversion script #408

Closed arbennett closed 4 years ago

arbennett commented 4 years ago

Mainly I don't like outputDefFile for aesthetic reasons, and typo-ability because of the fF portion. I'm fine with some other name as well, perhaps outputDefnFile? I think spatialParams is a clear improvement over trialParams - that has some implication that you'd try these and if they don't work do something else. However, this file is how you include spatially distributed parameters, hence spatialParams. The tables are an entirely separate problem to me, and I really don't like that they are implicit in SUMMA setups, but that's for another time.

Just one other thought, I personally think it's pretty obvious that these should be files, given the keys don't say "path" anywhere. I would be more in favor of dropping theFile from all of these in order to make space for better naming. Thoughts on this?