Skip to content

[RF] RooRealVar: Allow ranges and binnings to be created explicitly as non-shared#21944

Merged
guitargeek merged 2 commits intoroot-project:masterfrom
will-cern:master-rrv-sharedBinnings
Apr 21, 2026
Merged

[RF] RooRealVar: Allow ranges and binnings to be created explicitly as non-shared#21944
guitargeek merged 2 commits intoroot-project:masterfrom
will-cern:master-rrv-sharedBinnings

Conversation

@will-cern
Copy link
Copy Markdown
Contributor

At the moment all ranges and binnings created through the usual methods are created as shared. There are use cases where I would like to create a non-shared range on a variable. These changes are proposed modifications to achieve this functionality.

@will-cern will-cern requested a review from guitargeek as a code owner April 17, 2026 16:11
@guitargeek guitargeek self-assigned this Apr 18, 2026
@guitargeek guitargeek changed the title RooRealVar: Allow ranges and binnings to be created explicitly as non-shared [RF] RooRealVar: Allow ranges and binnings to be created explicitly as non-shared Apr 18, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 20, 2026

Test Results

    22 files      22 suites   3d 7h 8m 29s ⏱️
 3 833 tests  3 832 ✅  1 💤 0 ❌
76 573 runs  76 555 ✅ 18 💤 0 ❌

Results for commit 3f0ff38.

♻️ This comment has been updated with latest results.

@guitargeek
Copy link
Copy Markdown
Contributor

Hi @will-cern, thanks for the PR! There are failures in the CI. Do they make sense to you? Let me know if you need help in understanding what is going on.

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!

@guitargeek guitargeek merged commit f6f1f9e into root-project:master Apr 21, 2026
29 of 30 checks passed
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.

2 participants