[backport 106X] PUID UL16 and UL16APV training and working points

[singh-ramanpreet]

PR description:

• This PR is a backport to 10_6_X release cycle.
• The original PR [PUID] Add UL16 and UL16APV training configs #33857 is already merged into master
• Changes similar to PUID UL18 backport #32872, allowing use of new UL16(APV) training via run2_miniAOD_devel
• Also need backport of merged UL16 and UL16APV trainings files with Summer20 samples cms-data/RecoJets-JetProducers#13 (V05-14-00) into cmsdist

PR validation:

• passes basic tests
• test with UL16 and UL16APV trainings files with Summer20 samples cms-data/RecoJets-JetProducers#13

FYI: @alefisico @camclean @marinakolosova @mariadalfonso

[cmsbuild]

A new Pull Request was created by @singh-ramanpreet (Ramanpreet Singh) for CMSSW_10_6_X.

It involves the following packages:

RecoJets/JetProducers

@perrotta, @jpata, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks.
@rappoccio, @jdolen, @yslai, @jdamgov, @ahinzmann, @nhanvtran, @gkasieczka, @clelange, @schoef, @mariadalfonso, @seemasharmafnal 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

[mariadalfonso]

test parameters:

pull_request = cms-data/RecoJets-JetProducers#13

[mariadalfonso]

please test

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-356194/15543/summary.html
COMMIT: 63ec6d6
CMSSW: CMSSW_10_6_X_2021-06-01-2300/slc7_amd64_gcc700
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/33940/15543/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 35
• DQMHistoTests: Total histograms compared: 3215543
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3215208
• DQMHistoTests: Total skipped: 334
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 34 files compared)
• Checked 143 log files, 29 edm output root files, 35 DQM output files
• TriggerResults: no differences found

[jpata]

+reconstruction

• direct backport of [PUID] Add UL16 and UL16APV training configs #33857, additionally protected with the modifier run2_miniAOD_devel
• no unexpected reco changes (none expected)
• it is my understanding that the RecoJets-JetProducers tag for 10_6_X can simply be bumped V05-13-00->V05-14-00 and no backport is needed for the data PR. The only difference is the same UL16 and UL16APV trainings files with Summer20 samples cms-data/RecoJets-JetProducers#13.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_10_6_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_0_X is complete. 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)

[jpata]

@cms-sw/operations-l2 just to avoid confusion, will the RecoJets-JetProducers tag bump V05-13-00->V05-14-00 for cms-sw/cmsdist:10_6_X happen automatically, or should someone open a PR?

[silviodonato]

@cms-sw/operations-l2 just to avoid confusion, will the RecoJets-JetProducers tag bump V05-13-00->V05-14-00 for cms-sw/cmsdist:10_6_X happen automatically, or should someone open a PR?

someone should make a backport of https://github.com/cms-sw/cmsdist/pull/6961/files

[jpata]

someone should make a backport of https://github.com/cms-sw/cmsdist/pull/6961/files

Just to be clear, do you mean a backport in https://github.com/cms-sw/cmsdist? Which branch there for 10_6_X? And I understand we don't need to do a backport in https://github.com/cms-data/RecoJets-JetProducers, correct?

[qliphy]

@jpata yes, a cmsdist backport to the branch of IB/CMSSW_10_6_X/gcc700 (to update the version of RecoJets-JetProducers in data/cmsswdata.txt)

[jpata]

Thank you for the clarification, I created the cmsdist PR: cms-sw/cmsdist#6977

[singh-ramanpreet]

Hi @mariadalfonso
In addition to this backport, do we also need UL16 to be backported to other version of cmssw?

[mariadalfonso]

Hi @mariadalfonso
In addition to this backport, do we also need UL16 to be backported to other version of cmssw?

@singh-ramanpreet master + 10_6 are enough, no need of more back ports

[qliphy]

+1