-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[hist] TPrincipal bounds check and format #15110
Conversation
Starting build on |
Test Results 12 files 12 suites 2d 6h 44m 8s ⏱️ Results for commit f8e9c78. ♻️ This comment has been updated with latest results. |
Build failed on ROOT-ubuntu2004/python3. Errors:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, thank you very much! Don't worry about the backport,
I'll take care of that once I have accumulated a few PRs for a bulk backport (like this we don't spam too many PRs).
Thanks a lot! Could you also add #14818 to the batch? |
This Pull request:
Changes or fixes:
Clarifies covariance matrix format
Adds safety-bounds-checking
See https://root-forum.cern.ch/t/tprincipal-fails-for-a-very-large-number-of-entries/58795
Checklist: