search

natcap / invest

InVEST®: models that map and value the goods and services from nature that sustain and fulfill human life.
Apache License 2.0
159 stars 65 forks source link

TaskGraph add_task has new optional param #275

Closed dcdenu4 closed 3 years ago

dcdenu4 commented 4 years ago

TaskGraph will soon have an additional argument in add_task of store_result=False.

Adds a store_result flag to add_task to conditionally store a result rather than do it by default. In turn, get will now raise an exception if invoked on a Task created where the store_result flag is False. This PR adds that functionality, tests to cover it, and an update to history.

https://github.com/natcap/taskgraph/pull/34

We should make sure that InVEST is up to date with this change when upgrading to a new TaskGraph release.

dcdenu4 commented 3 years ago

From grep it looks like the following models will need to add this new argument: