Update centrality table for 2018 and 2021 heavy ion MC

[christopheralanwest]

PR description:

This PR updates the centrality table for 2018 and "2021" heavy ion MC. An update to the data GT will be necessary at some point in the future, prior to the re-miniAOD of the HI data, but we will wait until the release and strategy for that re-miniAOD is available prior to performing that update. This is the plan that was agreed upon with HI experts in the AlCaDB meeting of July 27, 2020.

The GT diffs are as follows:

2018 heavy ion
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/112X_upgrade2018_realistic_HI_v1/112X_upgrade2018_realistic_HI_v2

2021 heavy ion
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/112X_mcRun3_2021_realistic_HI_v5/112X_mcRun3_2021_realistic_HI_v6

Attn: @mandrenguyen

PR validation:

Please see the presentations at the July 27, 2020 AlCa/DB meeting for details of the validation. In addition, a technical test was performed:

runTheMatrix.py -l limited,159.0 --ibeos

if this PR is a backport please specify the original PR and why you need to backport that PR:

This PR is not a backport but may later be backported, depending upon in which release the re-miniAOD will be performed.

[cmsbuild]

The code-checks are being triggered in jenkins.

[christopheralanwest]

test parameters
workflow = 159.0

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-30930/17337

• This PR adds an extra 12KB to repository

[cmsbuild]

A new Pull Request was created by @christopheralanwest for master.

It involves the following packages:

Configuration/AlCa

@cmsbuild, @pohsun, @christopheralanwest, @tocheng, @tlampen can you please review it and eventually sign? Thanks.
@makortel, @Martin-Grunewald, @mmusich, @fabiocos, @tocheng this is something you requested to watch as well.
@silviodonato, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

[christopheralanwest]

please test

[cmsbuild]

The tests are being triggered in jenkins.
Test Parameters:

• MATRIX_EXTRAS = 159.0
• CMSSW_11_2_X_2020-07-27-1100/slc7_amd64_gcc820: https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/8323/console Started: 2020/07/27 22:03

[mandrenguyen]

For info, no change is expected from this GT update, which adds tags that are no yet consumed by any workflow.
A subsequent PR will activate in the HI miniAOD worklow the module RecoHI/HiCentralityAlgos/python/CentralityBin_cfi.py, which consumes the tags that are updated in this PR.

[cmsbuild]

+1
Tested at: 7ce8966
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1599a0/8323/summary.html
CMSSW: CMSSW_11_2_X_2020-07-27-1100
SCRAM_ARCH: slc7_amd64_gcc820

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1599a0/8323/summary.html

@slava77 comparisons for the following workflows were not done due to missing matrix map:

• /data/cmsbld/jenkins/workspace/compare-root-files-short-matrix/data/PR-1599a0/159.0_HydjetQ_B12_5020GeV_2021_ppReco+HydjetQ_B12_5020GeV_2021_ppReco+DIGIHI2021PPRECO+RECOHI2021PPRECO+ALCARECOHI2021PPRECO+HARVESTHI2021PPRECO

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 7 differences found in the comparisons
• DQMHistoTests: Total files compared: 34
• DQMHistoTests: Total histograms compared: 2525444
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2525396
• DQMHistoTests: Total skipped: 47
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 33 files compared)
• Checked 144 log files, 17 edm output root files, 34 DQM output files

[christopheralanwest]

+1

• Changes in slimmedTaus distributions for 2026 workflow 28234.0 are seen in other PRs, for example: Fix break condition #30926 (comment). I suppose this will be fixed once fix index typo in PFRecoTauClusterVariables #30919 is included in the IB.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next 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)

[silviodonato]

+1