-
Notifications
You must be signed in to change notification settings - Fork 7
Fixed parallel voxelization flag #101
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 @civanch for branch cms/v11.4.0. @akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
thanks @civanch , so this is not to fix the hanging tests? Note that relvals/addon and unit tests still hangs for Geant4 11.4 cms-sw/cmssw#49615 (comment) ( even with your fix in cmssw : cms-sw/cmssw#49615) |
|
@smuzaffar , if we exclude cms-sw/cmssw#49615? |
|
please test for CMSSW_16_0_GEANT4_X lets tests it without cms-sw/cmssw#49615 |
|
please test for CMSSW_16_0_G4VECGEOM_X |
|
please test with cms-sw/cmsdist#10243 for CMSSW_16_0_GEANT4_X @civanch , looks like tests did not hang without cms-sw/cmssw#49615. Let me re-run the tests with cmsdist PR which also has new geant4 data |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d94d74/49978/summary.html 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: |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d94d74/49976/summary.html 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: Comparison SummarySummary:
|
|
+externals thesis for special Geant4 IBs |
|
This pull request is fully signed and it will be integrated in one of the next cms/v11.4.0 IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @mandrenguyen, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d94d74/49978/summary.html 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: Comparison SummarySummary:
|
CMS unit tests crash if the default value of the parallel initialization flag is used. This PR fix the issue.