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.

Sign up for GitHub

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

Jump to bottom

[RF] Avoid setting out-of-range value in NaNPacker tests #18019

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[RF] Avoid setting out-of-range value in NaNPacker tests #18019

wants to merge 1 commit into from

Conversation

guitargeek
Copy link
Contributor

This would clip to the nearest in-range value anyway, so we should pass the lower limit directly.

@guitargeek
[RF] Avoid setting out-of-range value in NaNPacker tests
791912c 
This would clip to the nearest in-range value anyway, so we should pass
the lower limit directly.
@guitargeek guitargeek self-assigned this Mar 16, 2025
@guitargeek guitargeek requested a review from lmoneta as a code owner March 16, 2025 16:21
@github-actions GitHub Actions
Copy link

Test Results

    20 files      20 suites   5d 12h 58m 18s ⏱️
 2 726 tests  2 720 ✅ 0 💤 6 ❌
52 595 runs  52 589 ✅ 0 💤 6 ❌

For more details on these failures, see this check.

Results for commit 791912c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lmoneta lmoneta Awaiting requested review from lmoneta lmoneta is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@guitargeek guitargeek

Labels
in:RooFit/RooStats
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@guitargeek