MINIAOD: sorting of packedPFcandidates and PV association information - 74X same as #8132

[arizzi]

The information about which PV is matched to a candidate is added and the existing bits (fromPV) used for different definitions of CHS are improved to carry more information and allow change-of-PV from miniAOD.

In addition a sorting of the candidates is provided to improve the zlib compression (put similar things sequentially).

Unused ifdef'ed code is also removed.

Overall, this PR reduces the miniaod size by 2Kb/ev.
http://arizzi.web.cern.ch/arizzi/3bits.html (this PR)
vs
http://arizzi.web.cern.ch/arizzi/refib.html (original)

Legacy behaviour of PackedCandidate::fromPV() is implemented and gives 99.95% the same result (remaining difference can probably be fixed, if needed, by further tuning the accessor)

74X version of #8132

@gpetruc

[cmsbuild]

A new Pull Request was created by @arizzi for CMSSW_7_4_X.

MINIAOD: sorting of packedPFcandidates and PV association information - 74X same as #8132

It involves the following packages:

DataFormats/PatCandidates
PhysicsTools/PatAlgos

@cmsbuild, @vadler, @nclopezo, @monttj can you please review it and eventually sign? Thanks.
@rappoccio, @imarches, @ahinzmann, @acaudron, @mmarionncern, @jdolen, @nhanvtran, @schoef, @ferencek, @mariadalfonso, @pvmulder, @TaiSakuma 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.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[cmsbuild]

+1
Tested at: 942798a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8133/2345/summary.html

[cmsbuild]

Pull request #8133 was updated. @vadler, @monttj can you please check and sign again.

[cmsbuild]

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

[cmsbuild]

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

[arizzi]

can this be merged? (as it twin #8132 was)

[arizzi]

@davidlange6 can we get in sync with 75X? otherwise it is a mess to keep new PRs in sync..

[davidlange6]

@arizzi - would be better to develop in 75x and then port in chunks... 74x will tend to be behind 75x.

[davidlange6]

+1

[arizzi]

this PR is already integrated in 75X... other people are developing on 74X

[arizzi]

(closed by mistake)