JuliaIntervals / ValidatedNumerics.jl

Rigorous floating-point calculations with interval arithmetic in Julia
Other
85 stars 17 forks source link

Redo bisect #225

Closed dpsanders closed 7 years ago

dpsanders commented 7 years ago

Redoing #217 that was reverted in #223

codecov-io commented 7 years ago

Codecov Report

Merging #225 into master will increase coverage by 0.06%. The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
+ Coverage   91.91%   91.97%   +0.06%     
==========================================
  Files          24       25       +1     
  Lines        1026     1034       +8     
==========================================
+ Hits          943      951       +8     
  Misses         83       83
Impacted Files Coverage Δ
src/root_finding/root_finding.jl 85.1% <ø> (ø) :white_check_mark:
src/root_finding/bisect.jl 100% <100%> (ø)
src/intervals/intervals.jl 100% <100%> (ø) :white_check_mark:
src/decorations/intervals.jl 84.61% <0%> (ø) :white_check_mark:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a186a19...04170f3. Read the comment docs.

dpsanders commented 7 years ago

Do not merge for the moment.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.06%) to 91.973% when pulling 04170f3a991ae95cb35cb12686a09977b1d647cd on bisect into a186a19d22c5a221421eed2a409efd7b69c225dd on master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.06%) to 91.973% when pulling 04170f3a991ae95cb35cb12686a09977b1d647cd on bisect into a186a19d22c5a221421eed2a409efd7b69c225dd on master.

lbenet commented 7 years ago

Merging #227 I think this is automatically taken into account, am I right?

dpsanders commented 7 years ago

I included #227 in here to make sure that it did fix the problem. I had reverted the bisection PR, so this PR will need to be merged once #227 is.