-
Notifications
You must be signed in to change notification settings - Fork 60
add ITP bracketing method #274
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #274 +/- ##
==========================================
+ Coverage 83.82% 84.15% +0.33%
==========================================
Files 12 12
Lines 2015 2064 +49
==========================================
+ Hits 1689 1737 +48
- Misses 326 327 +1
Continue to review full report at Codecov.
|
src/bracketing.jl
Outdated
|
|
||
| x₁₂ = (a+b)/2 # __middle(a,b) | ||
| r = ϵ2n₁₂ / 2^j - (b-a)/2 | ||
| δ = κ₁ * (b - a)^κ₂ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On 1.8 this will be kind of fast if k_2 is an integer, but hard-coding k_2 to 2 would still be faster.
|
Thank you for putting together this PR so quickly - it looks great, and I am thrilled to have my ITP implementation in such an established package <3 |
cf discourse
To match Wikipedia's example: