Add resolution and pull plots to vertex validation, and fix two bugs

[makortel]

This PR adds the resolution (x, y, z, sum(pT^2); 1D and as a function of number of tracks and PU) and the pull (x, y, z) plots to the primary vertex validation. The definitions of the plots is similar to what we have for tracks in MultiTrackValidator. There are separate plots for cases when a reco vertex matches to one sim vertex and when it matches to many sim vertices (merged).

In addition, the following bugs are fixed

• Filling of KindOfSignalPV histogram when the signal PV is not the one with largest sum(pT^2)
  • Related to this, the setting of the kind_of_signal_vertex was simplified (in a separate commit to make it easier to verify that the results don't change)
• Setting the DQM folder for process.postProcessorHLTvertexing
  • In practice this affected only the standalone running of the postProcessorVertexStandalone sequence. In production workflows the process.postProcessorVertex did the post-processing for HLT, and process.postProcessorHLTvertexing did the post-processing for offline.

The first fix results the following change in 25ns RelValTTbar (9k events) (black old, red new)

i.e. events migrate from the bin !Highest!Assoc2Any to !HighestAssoc2First and !HighestAssoc2!First.

Tested in CMSSW_7_5_X_2015-02-25-1400.

@rovere @VinInn

[cmsbuild]

A new Pull Request was created by @makortel (Matti Kortelainen) for CMSSW_7_5_X.

Add resolution and pull plots to vertex validation, and fix two bugs

It involves the following packages:

Validation/RecoVertex

@cmsbuild, @danduggan, @nclopezo, @deguio can you please review it and eventually sign? Thanks.
@GiacomoSguazzoni, @rovere, @VinInn, @appeltel, @cerati, @dgulhan, @venturia this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[deguio]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: 879ca97
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8009/3058/summary.html

[cmsbuild]

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

[deguio]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar

[davidlange6]

+1