PecanProject / pecan

The Predictive Ecosystem Analyzer (PEcAn) is an integrated ecological bioinformatics toolbox.
www.pecanproject.org
Other
202 stars 235 forks source link

Add optional `pfts` argument to `run.sensitivity.analysis()` (fix/workaround for #3154) #3155

Closed Aariq closed 1 year ago

Aariq commented 1 year ago

Description

Adds a pfts argument to run.sensitivity.analysis() to allow the user to specify a subset of PFTs in the settings object to run sensitivity analysis on.

Also prevents this function from dumping objects into the global environment (and possibly overwriting things without any warning) by specifying the environment for load()

Motivation and Context

Doing sensitivity analysis with a large number of PFTs generates a large number of runs and to save on compute time, I "manually" removed SA runs for PFTs I'm not interested in. run.sensitivity.analysis() interprets this as an there having been an error in model runs and doesn't generate all the files it is supposed to (in particular, no sensitivity.results.Rdata) (#3154).

Review Time Estimate

Types of changes

Checklist: