Skip to content

rewrite integrator #80

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

rewrite integrator #80

wants to merge 9 commits into from

Conversation

mileslucas
Copy link
Collaborator

  • switch logvol and logwt calculations to use quadratic trapezoid rule
  • incorporate quad. trap. rule to logzerr calculation
  • add quadratic trapezoidal rule to add_live_points
  • rerun benchmarks with new integrator

@codecov
Copy link

codecov bot commented Jan 16, 2022

Codecov Report

Merging #80 (5667ae0) into main (eb6999e) will decrease coverage by 0.16%.
The diff coverage is 96.66%.

@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
- Coverage   88.62%   88.46%   -0.17%     
==========================================
  Files          13       11       -2     
  Lines         554      546       -8     
==========================================
- Hits          491      483       -8     
  Misses         63       63              
Impacted Files Coverage Δ
src/step.jl 94.77% <96.66%> (+1.12%) ⬆️
src/proposals/Proposals.jl 91.25% <0.00%> (-1.74%) ⬇️
src/bounds/Bounds.jl 84.61% <0.00%> (-0.57%) ⬇️
src/bounds/ellipsoid.jl 85.22% <0.00%> (-0.17%) ⬇️
src/NestedSamplers.jl
src/models/Models.jl
src/bounds/multiellipsoid.jl 79.62% <0.00%> (+1.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 eb6999e...5667ae0. Read the comment docs.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant