Skip to content

Conversation

cburgard
Copy link
Contributor

This Pull request:

Adds the ability to control the ToyMCSampler to emit binned toys.
Makes use of this new capability in the HypoTestCalculator in cases where the observed data is binned.

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

This PR fixes #

@cburgard cburgard changed the title [RF] ToyMCSampler improvements for binned toys [RF] ToyMCSampler improvements for binned toys [WIP] Oct 22, 2025
@cburgard
Copy link
Contributor Author

This PR is still WIP. This is because the use case relies on #20097 being merged first (even though the functionalities are not closely related).

@guitargeek guitargeek marked this pull request as draft October 22, 2025 16:08
@guitargeek guitargeek self-assigned this Oct 22, 2025
@guitargeek
Copy link
Contributor

I have merged #20097. Could you please rebase this PR?

Also, could you please add a commit where you add both of you to the list of contributors for ROOT 6.38 (insert in alphabetical order)?
https://github.com/root-project/root/blob/master/README/ReleaseNotes/v638/index.md

Thanks a lot!

@github-actions
Copy link

Test Results

    22 files      22 suites   3d 19h 35m 46s ⏱️
 3 692 tests  3 684 ✅ 0 💤   8 ❌
79 273 runs  79 097 ✅ 0 💤 176 ❌

For more details on these failures, see this check.

Results for commit fbb3468.

@cburgard cburgard force-pushed the toymcsampler-bugfix branch from fbb3468 to c5dddab Compare October 23, 2025 09:19
@cburgard cburgard force-pushed the toymcsampler-bugfix branch from c5dddab to 6dbedd7 Compare October 23, 2025 09:23
@cburgard
Copy link
Contributor Author

I have merged #20097. Could you please rebase this PR?

Also, could you please add a commit where you add both of you to the list of contributors for ROOT 6.38 (insert in alphabetical order)? https://github.com/root-project/root/blob/master/README/ReleaseNotes/v638/index.md

Thanks a lot!

Thanks a lot!

Done and done. I've removed the WIP marker now.

@cburgard cburgard changed the title [RF] ToyMCSampler improvements for binned toys [WIP] [RF] ToyMCSampler improvements for binned toys Oct 23, 2025
@cburgard cburgard marked this pull request as ready for review October 23, 2025 09:25
@cburgard cburgard requested a review from dpiparo as a code owner October 23, 2025 09:25
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