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

AVR-based CSVv2 taggers, updated SV TagInfos in MiniAOD #7787

Conversation

cmsbuild
Copy link
Contributor

This PR adds CSVv2 taggers based on secondary vertices reconstructed using the Adaptive Vertex Reconstructor (Run 1 default). PF-based version of CSVv2+AVR is also added to the standard reco. All new taggers were also propagated to PAT. In addition, SV TagInfos from which the SV info is grabbed and stored in MiniAOD has been updated.

To add a little bit of context. These changes are result of a discussion in yesterday's BTV POG meeting.
Automatically ported from CMSSW_7_4_X #7678

@cmsbuild
Copy link
Contributor Author

A new Pull Request was created by @cmsbuild for CMSSW_7_5_X.

AVR-based CSVv2 taggers, updated SV TagInfos in MiniAOD

It involves the following packages:

PhysicsTools/PatAlgos
RecoBTag/Configuration
RecoBTag/SecondaryVertex

@nclopezo, @cvuosalo, @monttj, @cmsbuild, @slava77, @vadler can you please review it and eventually sign? Thanks.
@rappoccio, @imarches, @ahinzmann, @acaudron, @mmarionncern, @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.
@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
Copy link
Contributor

+1
Tests already approved in the equivalent of this PR in CMSSW_7_4_X

@slava77
Copy link
Contributor

slava77 commented Feb 17, 2015

+1

for #7787 5e6882b
this is the same topic branch and the same commit ID as signed in

#7678 5e6882b

@cmsbuild why is there "orp-pending" signature?
75X PRs shouldn't have one for some time.

@ktf
Copy link
Contributor

ktf commented Feb 17, 2015

We (well, @davidlange6 ;)) do not want to fully open 75X, yet.

davidlange6 added a commit that referenced this pull request Feb 20, 2015
…efe5cd45436fca02bcfd36ba97a

AVR-based CSVv2 taggers, updated SV TagInfos in MiniAOD
@davidlange6 davidlange6 merged commit 146337b into cms-sw:CMSSW_7_5_X Feb 20, 2015
@ferencek ferencek deleted the CSVv2AVR_from-a332248d6e1f9efe5cd45436fca02bcfd36ba97a branch February 23, 2015 15:35
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.

None yet

6 participants