-
-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
More trust region schemes #151
Comments
Hi, @YingboMa , I am working on this issue. In Yuan's paper there are three different strategies laid out to update the trust region radius (2.10, 2.12 and 2.14). Which one should be used here? I ask this because these methods are locally superlinearly convergent but nothing is said about global convergence in the paper. |
Implement them as different options. I wouldn't trust their benchmarking too much and we should benchmark it ourselves in real contexts |
Okay! and by different options, do you mean implementing the |
Yes, so the |
Handled in #210 |
The current scheme for trust region radius selection will force the algorithm to stuck at local minima as shown in #146
New trust region radius calculations are suggested in:
https://www.tandfonline.com/doi/full/10.1080/10556788.2014.932943
https://link.springer.com/article/10.1007/s10107-015-0893-2#Sec4
The text was updated successfully, but these errors were encountered: