update to evtgen 2.0.0 and requirements to hepmc 2.06.10, photospp 3…

[alberto-sanchez]

This mean to update EvtGen to 2.0.0 and dependencies:

hepmc 2.06.10,
photospp 3.64,
and tauolpp 1.1.8

[cmsbuild]

A new Pull Request was created by @alberto-sanchez (Alberto Sanchez Hernandez) for branch IB/CMSSW_11_3_X/master.

@cmsbuild, @smuzaffar, @mrodozov can you please review it and eventually sign? Thanks.
cms-bot commands are listed here

[alberto-sanchez]

Not quite ready yet. we need to update update

cms-externals/evtgen#1

first

[cmsbuild]

Pull request #6494 was updated.

[cmsbuild]

Pull request #6494 was updated.

[alberto-sanchez]

please test

[alberto-sanchez]

ignore build-warnings

[cmsbuild]

-1

Failed Tests: Build
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-093149/11613/summary.html
CMSSW: CMSSW_11_3_X_2020-12-14-1100/slc7_amd64_gcc900

Build

I found compilation error when building:

>> Checking EDM Class Version for src/SimMuon/Neutron/src/classes_def.xml in libSimMuonNeutron.so
@@@@ ----> OK  EDM Class Version 
>> Checking EDM Class Transients in libRecoMuonMuonIdentification.so
@@@@ ----> OK  EDM Class Version 
>> Checking EDM Class Transients in libSimDataFormatsCrossingFrame.so
error: class 'HepMC::HeavyIon' has a different checksum for ClassVersion 10. Increment ClassVersion to 11 and assign it to checksum 1386021189
Suggestion: You can run 'scram build updateclassversion' to generate src/SimDataFormats/GeneratorProducts/src/classes_def.xml.generated with updated ClassVersion
gmake: *** [tmp/slc7_amd64_gcc900/edm_checks/libSimDataFormatsGeneratorProducts.so] Error 1
>> Checking EDM Class Version for src/SimTracker/TrackHistory/src/classes_def.xml in libSimTrackerTrackHistory.so
@@@@ ----> OK  EDM Class Version 
>> Checking EDM Class Transients in libSimDataFormatsJetMatching.so


[alberto-sanchez]

@smuzaffar , @mrodozov , anything I can do from my side to fix this warnings which apparently does not belong to this PR

[alberto-sanchez]

+1

[qliphy]

please test

[alberto-sanchez]

please test with cms-sw/cmssw#32452

[cmsbuild]

-1

Failed Tests: Build
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-093149/11915/summary.html
CMSSW: CMSSW_11_3_X_2021-01-02-1100/slc7_amd64_gcc900

Build

I found compilation error when building:

@@@@ ----> OK  EDM Class Version 
>> Checking EDM Class Transients in libRecoPixelVertexingPixelTriplets.so
>> Checking EDM Class Transients in libSimDataFormatsCrossingFrame.so
@@@@ ----> OK  EDM Class Version 
>> Checking EDM Class Transients in libRecoMuonMuonIdentification.so
error: class 'HepMC::HeavyIon' has a different checksum for ClassVersion 10. Increment ClassVersion to 11 and assign it to checksum 1386021189
@@@@ ----> OK  EDM Class Transients 
Suggestion: You can run 'scram build updateclassversion' to generate src/SimDataFormats/GeneratorProducts/src/classes_def.xml.generated with updated ClassVersion
>> Checking EDM Class Version for src/SimMuon/DTDigitizer/test/stubs/classes_def.xml in libtestSimMuonDTDigitizer.so
gmake: *** [tmp/slc7_amd64_gcc900/edm_checks/libSimDataFormatsGeneratorProducts.so] Error 1
>> Checking EDM Class Version for src/SimMuon/Neutron/src/classes_def.xml in libSimMuonNeutron.so


[mrodozov]

@alberto-sanchez this needs update on the CMSSW side.
see this for example:
cms-sw/cmssw#32420
if you see the the error scram tells you to checkout the package containing HepMC::HeavyIon
and change it's class version to new version

[mrodozov]

ah you already did. let me see

[mrodozov]

test parameters:

• pull_requests=Update to evtgen 2.0 cmssw#32452
• release_queue=CMSSW_11_3_X_2021-01-03-2300/slc7_amd64_gcc900

[mrodozov]

please test

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-093149/11922/summary.html
CMSSW: CMSSW_11_3_X_2021-01-03-2300/slc7_amd64_gcc900

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 2 differences found in the comparisons
• DQMHistoTests: Total files compared: 37
• DQMHistoTests: Total histograms compared: 2716967
• DQMHistoTests: Total failures: 7
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2716938
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 36 files compared)
• Checked 156 log files, 37 edm output root files, 37 DQM output files

[mrodozov]

+externals
@qliphy lets merge this and cms-sw/cmssw#32420 together on your request
@alberto-sanchez would this work without 32420 ? If yes, I can merge it first otherwise they go together

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_11_3_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2)

[alberto-sanchez]

@mrodozov, yes that should work without 32420. In 32452 I am updating the version for HepMC::HeavyIon, which
is the only required for this PR.

[mrodozov]

I am updating the version for HepMC::HeavyIon

that will fail if I merge this. @qliphy merge them both when ready

[qliphy]

In 32452 I am updating the version for HepMC::HeavyIon, which
is the only required for this PR.

@alberto-sanchez This has been done with your commit: cms-sw/cmssw@965f6c6
Right? If so I think we can merge both PRs.

[alberto-sanchez]

That is correct.

[qliphy]

+1