Bugfix: Set Photon ID/vertex information for PFlow selected photons.

[lgray]

Set mva value to one as well as primary vertex for photons selected by PFlow.

Shouldn't have an effect on gedPhotons, only changes plots that analyze directly the particleFlow:photons collection.

[cmsbuild]

A new Pull Request was created by @lgray (Lindsey Gray) for CMSSW_7_0_X.

Bugfix: Set Photon ID/vertex information for PFlow selected photons.

It involves the following packages:

RecoParticleFlow/PFProducer

@nclopezo, @cmsbuild, @thspeer, @slava77 can you please review it and eventually sign? Thanks.
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.
@ktf you are the release manager for this.

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/1279/summary.html

[slava77]

+1

Tested in CMSSW_7_0_X_2013-11-14-0200 in a combination with

• bugfixes for PFEG: mustache conversions, GSF-SC associations, muons, mva-e-pi filling #1383 1714bd0
• Bugfix: Set Photon ID/vertex information for PFlow selected photons. #1396 ab5491b

Normal mode:

• increased rate of errors in _logErrorHarvester: in 202.0 18.0 and 38.0
  • e.g. in 202.0: 1 -> 4 of
    %MSG-w PFEGammaAlgo::mergeROsByAnyLink: PFEGammaProducer:particleFlowEGamma
    Encountered the known GSF-SC splitting bug in PFBlockAlgo! We should really fix this!
• minor variations in the ged photons and electrons and in egamma superclusters, consistent with changes in brem->cluster asociation.
• large drop of number of ged electrons reconstructed from high-pt muons (expected bugfix)
• mva values in ged electrons are now filled (only visible in the fwlite script)

GED mode:

• large drop of electrons reconstructed from high-pt muons (expected bugfix)
• (connected to above) tau validation plots with muon rejection are now ~100% efficient (as in the original taus)
• mva values for the electrons are now filled and show up in DQM plots as well

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests. @ktf can you please take care of it?