-
Notifications
You must be signed in to change notification settings - Fork 183
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
[G4][geant4_only] Update geant4 to 10.7.2 #7042
Conversation
Get the latest G4 tag in the G4 IB
please test |
A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_12_0_X/geant4. @smuzaffar, @mrodozov, @iarspider can you please review it and eventually sign? Thanks. |
-1 Failed Tests: Build 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: BuildI found compilation error when building: Entering library rule at src/RecoTracker/MkFit/plugins >> Compiling edm plugin /build/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_GEANT4_X_2021-06-15-2300/src/RecoTracker/MkFit/plugins/MkFitInputConverter.cc >> Compiling edm plugin /build/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_GEANT4_X_2021-06-15-2300/src/RecoTracker/MkFit/plugins/MkFitOutputConverter.cc >> Compiling edm plugin /build/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_GEANT4_X_2021-06-15-2300/src/RecoTracker/MkFit/plugins/MkFitProducer.cc /build/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_GEANT4_X_2021-06-15-2300/src/RecoTracker/MkFit/plugins/MkFitProducer.cc: In constructor 'MkFitProducer::MkFitProducer(const edm::ParameterSet&)': /build/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_GEANT4_X_2021-06-15-2300/src/RecoTracker/MkFit/plugins/MkFitProducer.cc:58:29: error: no match for 'operator=' (operand types are 'std::function' and 'double(mkfit::Event&, const mkfit::EventOfHits&, mkfit::MkBuilder&)') 58 | buildFunction_ = mkfit::runBuildingTestPlexBestHit; | ^~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from /cvmfs/cms-ib.cern.ch/nweek-02685/slc7_amd64_gcc900/external/gcc/9.3.0/include/c++/9.3.0/functional:59, from /cvmfs/cms-ib.cern.ch/week1/slc7_amd64_gcc900/cms/cmssw/CMSSW_12_0_GEANT4_X_2021-06-15-2300/src/DataFormats/Provenance/interface/Hash.h:5, from /cvmfs/cms-ib.cern.ch/week1/slc7_amd64_gcc900/cms/cmssw/CMSSW_12_0_GEANT4_X_2021-06-15-2300/src/DataFormats/Provenance/interface/ParentageID.h:5, |
@mrodozov , it is not clear why there is build error in RECO sub-library while only Geant4 version is changed. |
@civanch we don't have a fresh IB that includes this change in mkFit . This evening we'll have it and restart the test |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-308be7/16113/summary.html Comparison SummarySummary:
|
@mrodozov , I think the test is now positive, the fact that simulations histories are different and we have no regression means, that we have to do private validation and PPD validation campaign. This PR may be merged to special branches GEANT4 and G4VECGEOM. |
Get the latest G4 tag in the G4 IB