Skip to content
This repository was archived by the owner on Jun 18, 2025. It is now read-only.

Conversation

sritchie
Copy link
Member

@sritchie sritchie commented Feb 28, 2022

  • Add bisect namespace for bisection search #490 adds sicmutils.numerical.roots.bisect with implementations of bisection
    search, secant search and a mixed method found in scmutils. These all live
    under a bisect function.

    The data structure returned is similar to the minimization functions in the
    sicmutils.numeric.{unimin, multimin} namespaces. As more root-finding
    methods come online this should all standardize nicely.

@codecov-commenter
Copy link

Codecov Report

Merging #490 (874ed03) into main (45e251d) will decrease coverage by 0.09%.
The diff coverage is 71.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #490      +/-   ##
==========================================
- Coverage   86.38%   86.29%   -0.10%     
==========================================
  Files         100      101       +1     
  Lines       14414    14509      +95     
  Branches      750      758       +8     
==========================================
+ Hits        12452    12520      +68     
- Misses       1212     1231      +19     
- Partials      750      758       +8     
Impacted Files Coverage Δ
src/sicmutils/numerical/roots/bisect.cljc 71.57% <71.57%> (ø)
src/sicmutils/util/stream.cljc 89.58% <100.00%> (ø)
src/sicmutils/rational_function.cljc 80.33% <0.00%> (-0.85%) ⬇️
src/sicmutils/polynomial.cljc 83.43% <0.00%> (+0.15%) ⬆️
src/sicmutils/numsymb.cljc 92.68% <0.00%> (+0.44%) ⬆️

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 45e251d...874ed03. Read the comment docs.

@sritchie sritchie merged commit 53b1f8e into main Mar 1, 2022
@sritchie sritchie deleted the sritchie/bisect branch March 1, 2022 20:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants