Flavor bugfix in jetTools.py (74X)

[ferencek]

Backport of #11501

Update (Oct. 1, 2015): Included backport of #11612

[cmsbuild]

A new Pull Request was created by @ferencek (Dinko Ferencek) for CMSSW_7_4_X.

Flavor bugfix in jetTools.py (74X)

It involves the following packages:

PhysicsTools/JetExamples
PhysicsTools/JetMCAlgos
PhysicsTools/PatAlgos

@cmsbuild, @vadler, @monttj can you please review it and eventually sign? Thanks.
@TaiSakuma, @imarches, @ahinzmann, @acaudron, @mmarionncern, @rappoccio, @jdolen, @nhanvtran, @schoef, @ferencek, @gpetruc, @mariadalfonso, @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.
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.

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: fa2641f
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11503/8412/summary.html

[cmsbuild]

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

[cmsbuild]

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

[ferencek]

@monttj, could you please sign this PR. Thanks

[ferencek]

Now that there are no active MiniAOD campaigns in 74X, I suggest this PR gets merged.

[slava77]

@ferencek MC requests are still coming and being processed (so far with older release);
there is also a 0T reprocessing campaign about to start.
So, 74X is still pretty active.

[slava77]

@cmsbuild please test
(at least to have a record of the tests results)

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10497/console

[ferencek]

@slava77, thanks for the info. MC requests should be unaffected but MiniAOD for 0T reprocessing, on the other hand, would be affected. So I guess it is best to wait until all MiniAOD campaigns are completed.

[slava77]

sorry, 0T is in 76X.
MC injections are still ongoing though.

[slava77]

I think further plans for 74X can be discussed in the ORP (no more changes is still an easier choice).
IMO, if we let changes to happen, it will be more than just this PR.

[ferencek]

In that case, there is no problem with merging this PR since the only data tier it affects is MiniAOD. This and possibly other 74X PR would still be useful for some analysis-level work until a complete migration to 76X is made.

[cmsbuild]

+1
Tested at: 2d33db9
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11503/10497/summary.html

[cmsbuild]

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

[ferencek]

No longer relevant after 76X re-reco. Closing