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

[Minuit2] Code improvements #17869

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

guitargeek
Copy link
Contributor

Draft PR for now, to see what the CI says.

Copy link

github-actions bot commented Mar 3, 2025

Test Results

    15 files      15 suites   3d 11h 52m 53s ⏱️
 2 726 tests  2 725 ✅ 0 💤 1 ❌
39 541 runs  39 540 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit a8024ce.

♻️ This comment has been updated with latest results.

As suggested by the comment in the documentation.
Remove unused and controversial `X_pos` and `X_neg` functions from the
MnParabola.

Also, reduce the documentation a bit because it was too verbose for what
the class represents.
The `MnTiny` class was a tiny class that was only used inside
`MnMachinePrecision`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant