opendatastudio / bindfit-dataflow

Bindfit datapackage for opendata.studio
0 stars 0 forks source link

Merge options into params resource #10

Closed echus closed 1 year ago

echus commented 1 year ago

Fit options are tightly coupled to parameter spaces, so it makes sense to reference them inside the parameter resource that they apply to. This will allow us to specify different options & defaults for different parameter spaces (e.g. UV and NMR fitters in Bindfit).

echus commented 1 year ago

This requires https://github.com/opendatafit/execution-base/issues/16

echus commented 1 year ago

Decided not to wait on implementation of https://github.com/opendatafit/execution-base/issues/16, will hack around it for now by passing complete datapackage to all algorithms along with algorithm inputs and getting options from that. In the long run we will get rid of this when transitioning to Github datapackage management.