Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IVF cleanup + trackJetPt as TaggingVariable: Backport from 7_1_X to 7_0_X #3394

Conversation

ferencek
Copy link
Contributor

Backport of #2941 but without any changes to the RECO sequence

@cmsbuild
Copy link
Contributor

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

IVF cleanup + trackJetPt as TaggingVariable: Backport from 7_1_X to 7_0_X

It involves the following packages:

DataFormats/BTauReco
PhysicsTools/PatAlgos
RecoBTag/SecondaryVertex
RecoVertex/AdaptiveVertexFinder

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

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@slava77
Copy link
Contributor

slava77 commented Apr 22, 2014

+1

for #3394 a2e5edf

tested in CMSSW_7_0_X_2014-04-22-0200
(test area sign704a in a combination of pulls 3165 3254 3273 3394)
including extended tests in few-K particle gun samples, few-hundred dijet, and ttbar with PU.
No regressions observed in monitored RECO quantities,
no differences in all harvested DQM plots for the extended tests (timing module jitter ignored).

@vadler
Copy link

vadler commented May 8, 2014

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented May 8, 2014

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

@vadler
Copy link

vadler commented May 8, 2014

@davidlange6 : Could this please be merged as soon as possible, since #3667 is waiting for it to be updated accordingly?

arizzi added a commit to arizzi/cmssw that referenced this pull request May 9, 2014
davidlange6 added a commit that referenced this pull request May 9, 2014
…CMSSW_7_0_1

IVF cleanup + trackJetPt as TaggingVariable: Backport from 7_1_X to 7_0_X
@davidlange6 davidlange6 merged commit 6d22664 into cms-sw:CMSSW_7_0_X May 9, 2014
@ferencek ferencek deleted the IVFcleanup-trackJetPt_from-CMSSW_7_0_1 branch May 9, 2014 11:32
@vadler
Copy link

vadler commented May 9, 2014

Thanx, @davidlange6 :-)

gpetruc pushed a commit to gpetruc/cmssw that referenced this pull request May 9, 2014
gpetruc pushed a commit to gpetruc/cmssw that referenced this pull request May 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants