[10.1.X] Add switch to contrain vertex refit to BeamSpot in Alignment PV Validation

[mmusich]

backport of #23448

Greetings,
this PR is meant to introduce a new configuration switch in the Alignment Primary Vertex Validation to allow using the Beam Spot from DB information in the unbiased vertex refit procedure. By default this is switched to False, so in normal conditions this addition is completely transparent to the validation procedure.
This has been shown to be useful to cross-check the scrutiny of the Full-Track validation of the alignment payload at this Hypernews thread.
See here an example of output validation plot obtained for run 31766 either in standard conditions, forcing the vertex fit with the BS from DB prior to re-computation of BS with the new alignment and forcing the vertex fit with the updated BS.

I profit of this PR also to update the initialization file of the all-in-one meta-data tool adding links to the twiki documentation.

[cmsbuild]

A new Pull Request was created by @mmusich (Marco Musich) for CMSSW_10_1_X.

It involves the following packages:

Alignment/OfflineValidation

@cmsbuild, @franzoni, @arunhep, @cerminar, @lpernie can you please review it and eventually sign? Thanks.
@mschrode, @mmusich, @tocheng, @tlampen this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

• Backported from [10.2.X] Add switch to contrain vertex refit to BeamSpot in Alignment PV Validation #23448

[lpernie]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/28446/console Started: 2018/06/04 22:39

[cmsbuild]

+1
Tested at: e01a5b9
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23456/28446/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 29
• DQMHistoTests: Total histograms compared: 2498020
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2497843
• DQMHistoTests: Total skipped: 176
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 28 files compared)
• Checked 119 log files, 9 edm output root files, 29 DQM output files

[lpernie]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_10_1_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_10_2_X is complete. This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[fabiocos]

@mmusich this code has been integrated in master based on your validation, and does not give any obvious problem. As far as I can see it does not appear in standard production workflows, please correct me. BTW the input file used by the test looks still a 9_2_X relVal

[mmusich]

@fabiocos

As far as I can see it does not appear in standard production workflows, please correct me.

that's correct. This PR is mostly for convenience of the aligners to have a 10_1 release with it when validating the next object for production.

BTW the input file used by the test looks still a 9_2_X relVal,

do you mean that the test that runs in the unit tests uses:

from PhysicsTools.PatAlgos.patInputFiles_cff import filesRelValTTbarPileUpGENSIMRECO

?
This uses indeed 9_2_X relval file at the moment:

cmssw/PhysicsTools/PatAlgos/python/patInputFiles_cff.py

Line 61 in 4ee7a12

filesRelValTTbarPileUpGENSIMRECO = cms.untracked.vstring(

but it doens't really matter as long as the dataformat is not broken (I hope that people responsible for PhysicsTools.PatAlgos keep it reasonably updated as it used in several other unit tests in CMSSW).

[fabiocos]

+1