Skip to content
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

test: increase integration test tolerance #247

Merged
merged 2 commits into from
Aug 3, 2021

Conversation

alexander-held
Copy link
Member

@alexander-held alexander-held commented Aug 3, 2021

Slightly increases the tolerance for integration tests. This is required to make the tests pass with iminuit 2.8.0, which updated the minimizer code to the ROOT master. The resulting change in fit results is negligible and has no physical impact.

@codecov
Copy link

codecov bot commented Aug 3, 2021

Codecov Report

Merging #247 (87253ff) into master (90ac830) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #247   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           15        15           
  Lines         1716      1716           
  Branches       267       267           
=========================================
  Hits          1716      1716           

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 90ac830...87253ff. Read the comment docs.

@alexander-held alexander-held merged commit 313ba7b into master Aug 3, 2021
@alexander-held alexander-held deleted the test/integration-test-tolerance branch August 3, 2021 19:10
# 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