Second part of miniAOD developments

[arizzi]

This contains more developments in the context of miniAOD.
@lgray @gpetruc @ferencek

Contains IVF (btag) related code, ecal cluster shape stuff, updated pruned gen particles content.

Similar PR for 70X will follow once #3254 is integrated.

[cmsbuild]

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

Second part of miniAOD developments

It involves the following packages:

DataFormats/BTauReco
DataFormats/Candidate
DataFormats/PatCandidates
PhysicsTools/PatAlgos
RecoEcal/EgammaClusterProducers
RecoEcal/EgammaCoreTools

@nclopezo, @monttj, @cmsbuild, @anton-a, @thspeer, @slava77, @vadler, @Degano can you please review it and eventually sign? Thanks.
@TaiSakuma, @lgray, @nhanvtran, @ferencek, @argiro, @pvmulder 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.
@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.

[nclopezo]

Hi @arizzi,

This pull request is not mergeable, could you please rebase it?

[arizzi]

now merged to the current 71X and tested on CMSSW_7_1_X_2014-04-29-0200

[arizzi]

mhh I guess I did not rebase in the proper way... let me try again

[arizzi]

no luck, if I rebase I have to re-resolve all conflicting commits of last two weeks that me and giovanni already resolved in this branch PR.
If I merge myself I get a single merge commit with zillion changes, you can see for example the second last commit of:
arizzi#25

if such commit is not a problem I can update this PR, otherwise someone with better git skills can try to solve it in a better way

[nclopezo]

Hi @arizzi,
I only see conflicts in the file
PhysicsTools/PatAlgos/python/tools/jetTools.py
I did the following:

scram p CMSSW_7_1_X_2014-04-29-0200
cd CMSSW_7_1_X_2014-04-29-0200/
cmsenv
git cms-merge-topic 3545
cd src/
git status

You can follow the instructions in the following link to solve it:
http://cms-sw.github.io/cmssw/tutorial-forward-port.html

[arizzi]

Hi David,
I can merge and solve the conflict but then for some reason that I do not want to understand myself (I wasted already half a day to do so) the resulting commit is not just the jetTools.py fix but rather a largish commit.
Anyhow I do not care if this is not ok, I push the whole thing as you suggested even if the created commit is not minimal.

[cmsbuild]

Pull request #3545 was updated. @nclopezo, @monttj, @cmsbuild, @anton-a, @thspeer, @slava77, @vadler, @Degano can you please check and sign again.

[cmsbuild]

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

[slava77]

testing

[slava77]

+1

for #3545 e4b4ea6
tested in CMSSW_7_1_X_2014-05-05-1400 (test area sign368)

• nominal doubling of the code in EcalClusterTools is not really visible mostly because it's inlined now and not all functions are used
• there is a minor numerical precision level change in photon sigma_ietaieta plots (fwiw, essentially invisible)

[vadler]

This one should go to the tools directory at some point.

[vadler]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?