tjhladish / AbcSmc

Sequential Monte Carlo Approximate Bayesian Computation with Partial Least Squares parameter estimator
GNU General Public License v3.0
15 stars 7 forks source link

shifting functionality towards AbcUtil #29

Closed pearsonca closed 1 year ago

pearsonca commented 1 year ago

Generally desire:

To this end, transitioning "stateful" functions (that e.g. know about the whole AbcSmc state, rather than the particular mathematical objects relevant to some particular operation) off on AbcSmc object methods into AbcUtil/ABC namespace methods that are stateless. As this happens, reorganizing naming / arguments order / etc to introduce consistency.

Trying to maintiain the commits here such that this PR can be resolved whenever, but haven't grappled with the full scope / checklist of making the above happen.

pearsonca commented 1 year ago

Merge issues aside, satisfied this is complete. There are a few items which warrant relocation, but entail a slightly larger reorganization. Basically, changing from requiring the whole smc history => only the last / current states. That would generally eliminate the use of the smcSet argument / indicator, collapse several vectors of stuff to was/is fields.

I think it would roughly only change the storage requirements to add the weights.

pearsonca commented 1 year ago

dealt with merge fiddliness, continuing the i/j/k update in abcutil, and spacing issues