Skip to content

[RF][HS3] Fixed snapshot filter, unbinned data edges, zero yield bins#22073

Merged
guitargeek merged 3 commits intoroot-project:masterfrom
Phmonski:HS3patch
May 8, 2026
Merged

[RF][HS3] Fixed snapshot filter, unbinned data edges, zero yield bins#22073
guitargeek merged 3 commits intoroot-project:masterfrom
Phmonski:HS3patch

Conversation

@Phmonski
Copy link
Copy Markdown
Contributor

This Pull request:

Fixed snapshot filter, unbinned data edges, zero yield bins

Changes or fixes:

  • Snapshot export no longer filters global observables
  • Axes of unbinned datasets now contain information on the range of observables
  • Added a fail safe for histfactory bins with zero yield and non-zero error

Checklist:

  • tested changes locally

@cburgard

@Phmonski Phmonski requested a review from guitargeek as a code owner April 27, 2026 10:40
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

Test Results

    22 files      22 suites   3d 10h 0m 20s ⏱️
 3 847 tests  3 846 ✅ 0 💤 1 ❌
76 894 runs  76 893 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 5f6bedc.

♻️ This comment has been updated with latest results.

Phmonski and others added 3 commits May 7, 2026 23:13
  * Snapshot export no longer filters global observables
  * Axes of unbinned datasets now contain information on the range of
    observables
  * Added a fail safe for histfactory bins with zero yield and non-zero
    error
@guitargeek guitargeek changed the title [HS3] Fixed snapshot filter, unbinned data edges, zero yield bins [RF][HS3] Fixed snapshot filter, unbinned data edges, zero yield bins May 7, 2026
Copy link
Copy Markdown
Contributor

@guitargeek guitargeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much! I also have added a test and some general improvements.

@guitargeek guitargeek merged commit 16b2075 into root-project:master May 8, 2026
28 of 30 checks passed
@guitargeek
Copy link
Copy Markdown
Contributor

/backport to 6.40

@root-project-bot
Copy link
Copy Markdown

Preparing to backport PR #22073 to branch 6.40 requested by guitargeek

@root-project-bot
Copy link
Copy Markdown

This PR has been backported to branch 6.40: #22184

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.

3 participants