-
Notifications
You must be signed in to change notification settings - Fork 208
updated geant4 version 11.4.0 #10243
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
updated geant4 version 11.4.0 #10243
Conversation
|
please test |
|
A new Pull Request was created by @raoatifshad for branch IB/CMSSW_16_0_X/g4. @akritkbehera, @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
please test |
|
please test for CMSSW_16_0_GEANT4_X |
|
please test for CMSSW_16_0_G4VECGEOM_X |
|
-1 Failed Tests: UnitTests RelVals AddOn 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 11 errors in the following unit tests: ---> test test_MC_22_crosscheck had ERRORS ---> test test_MC_23_crosscheck had ERRORS ---> test trackerMaterialAnalysisPlots had ERRORS and more ... Failed RelValsThe relvals timed out after 4 hours. Failed AddOn Tests
|
|
-1 Failed Tests: UnitTests RelVals AddOn 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 11 errors in the following unit tests: ---> test materialBudgetHGCalPlots had ERRORS ---> test materialBudgetTrackerPlots had ERRORS ---> test testTauEmbeddingWorkflow2016postVFP had ERRORS and more ... Failed RelValsThe relvals timed out after 4 hours. Failed AddOn Tests
|
|
Unit tests have timed out. I have restarted tests job for G4, let's see if the issue persists. |
|
Yes, the issue persists. @civanch FYI |
|
-1 Failed Tests: UnitTests RelVals AddOn 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 11 errors in the following unit tests: ---> test test_MC_22_crosscheck had ERRORS ---> test test_MC_23_crosscheck had ERRORS ---> test trackerMaterialAnalysisPlots had ERRORS and more ... Failed RelValsThe relvals timed out after 4 hours. Failed AddOn Tests
|
|
please test for CMSSW_16_0_G4VECGEOM_X lets see if full cmssw works |
|
@civanch can you provide the relevant code change in |
|
@kpedro88 , you are right. may be we will do that if will not find out a solution for unit tests. |
|
@civanch , Can you please open a PR against https://github.com/cms-externals/geant4/tree/cms/v11.4.0 for your fix/change? |
|
@smuzaffar , will do. |
|
@iarspider , @smuzaffar , PR cms-externals/geant4#101 is done |
|
REMINDER @sextonkennedy, @mandrenguyen, @ftenchini: This PR was tested with cms-externals/geant4#101, please check if they should be merged together |
|
Pull request #10243 was updated. |
|
please test for CMSSW_16_0_G4VECGEOM_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 10 errors in the following unit tests: ---> test TestFWCoreIntegrationStandalone had ERRORS ---> test unitTestsGroup_1_u2 had ERRORS ---> test unitTestsGroup_4_u20 had ERRORS and more ... Comparison SummarySummary:
|
|
please test for CMSSW_16_0_G4VECGEOM_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 10 errors in the following unit tests: ---> test TestFWCoreIntegrationStandalone had ERRORS ---> test unitTestsGroup_1_u2 had ERRORS ---> test unitTestsGroup_4_u20 had ERRORS and more ... Comparison SummarySummary:
|
|
+externals unit test failures are not due to this change |
|
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_16_0_X/g4 IBs (but tests are reportedly failing). 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) |
Updated the geant4 version 11.4.0
updated the dataset PhotonEvaporation-6.1.2