-
Notifications
You must be signed in to change notification settings - Fork 0
added missing header for assert #1
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
Conversation
|
A new Pull Request was created by @smuzaffar for branch cms/v44.0.1. @akritkbehera, @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
please test for CMSSW_16_0_PY312_X |
|
-1 Failed Tests: UnitTests The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Failed Unit TestsI found 1 errors in the following unit tests: ---> test test-das-selected-lumis had ERRORS Comparison SummarySummary:
|
|
@smuzaffar can you make a PR for the upstream version of Acts at https://github.com/acts-project/acts/ ? Otherwise we end up maintaining a stack of patches, fixing conflicts, rebasing, etc. |
|
@fwyzard, I opened acts-project#4768 |
|
Thanks ! |
|
I've opened cms-sw/cmsdist#10172 to use the same commit as in your PR in our branch. |
FYI @fwyzard
acts failed to build for PY312 IBs which has newer TF/eigen and other tools