Issue
Some of the defaults can be updated to be the latest configuration.
- We have suitable evidence that
Unscented and TransformUnscented can use acceleration and it is beneficial - therefore making a good default
- We should note infinite-time algorithms [
Inversion(prior), TransformInversion(prior) , Unscented, and others, do not have a "special" time T=1 at which they should be terminated early. Therefore the final time should be extended.
impose_prior = true is our most common Unscented and TransformUnscented setting and so should be made a default.
Suggested solution:
- Add
NesterovAccelerator(), (ekp default) and change impose_prior=true (uki/utki default) for Unscented and TransformUnscented variants
- Replace
DataMisfitController(terminate_at=1) with DataMisfitController(terminate_at=1e8) for infinite-time algorithms
Issue
Some of the defaults can be updated to be the latest configuration.
UnscentedandTransformUnscentedcan use acceleration and it is beneficial - therefore making a good defaultInversion(prior),TransformInversion(prior),Unscented, and others, do not have a "special" timeT=1at which they should be terminated early. Therefore the final time should be extended.impose_prior = trueis our most commonUnscentedandTransformUnscentedsetting and so should be made a default.Suggested solution:
NesterovAccelerator(), (ekp default) and changeimpose_prior=true(uki/utki default) forUnscentedandTransformUnscentedvariantsDataMisfitController(terminate_at=1)withDataMisfitController(terminate_at=1e8)for infinite-time algorithms