-
Notifications
You must be signed in to change notification settings - Fork 53
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
muller inefficient #221
Comments
jacobwilliams
added a commit
to jacobwilliams/Roots.jl
that referenced
this issue
Jul 17, 2021
Just sent a PR for this. I haven't looked at any of the other methods to see if they have the same problem. |
Thanks for this! I'll merge in and bump the version number once the tests run. |
jverzani
pushed a commit
that referenced
this issue
Jul 18, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
muller
function is very inefficient since it's recomputing all three function evaluations every iteration, even though two of them have already been computed in the previous iteration.The text was updated successfully, but these errors were encountered: