tensorflow / probability

Probabilistic reasoning and statistical analysis in TensorFlow
https://www.tensorflow.org/probability/
Apache License 2.0
4.25k stars 1.1k forks source link

Allow optional initial inverse Hessian scaling in bfgs.minimize #1756

Closed gmjw closed 1 year ago

gmjw commented 1 year ago

This aims to address issue #1754

As stated in that issue, previous code did not allow for clean iterative use of the BFGS algorithm. I would hope that this setup does.

I've tried to show in the added test that the option is having its intended effect.

google-cla[bot] commented 1 year ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.