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
Trivial Fix for VertexFromTrackProducer #4263
Trivial Fix for VertexFromTrackProducer #4263
Conversation
A new Pull Request was created by @battibass (Carlo Battilana) for CMSSW_7_1_X. Trivial Fix for VertexFromTrackProducer It involves the following packages: RecoTauTag/HLTProducers @perrotta, @cmsbuild, @nclopezo, @Martin-Grunewald, @fwyzard, @Degano can you please review it and eventually sign? Thanks. |
+1 |
1 similar comment
Trivial Fix for VertexFromTrackProducer
This is a trivial fix for a bug introduced during the "consumes migration" of VertexFromTrackProducer.
Basically the bug was affecting the rate reduction power of isolation @ HLT.
Tracks from edm:Vievreco::RecoCandidate > were not properly accessed, preventing from correct generation of vertexes from RecoCandidate tracks.
The net effect in rate reduction for a sample (IsoMu24) path can be summarized as:
Release details / Relative Rate w.r.t. non Iso Path
CMSSW_7_1_0_pre5 (last pre release before the bug was introduced) / relative rate: ~37%
CMSSW_7_1_0_pre9 (before bugfix) / relative rate: ~45%
CMSSW_7_1_0_pre9 (after bugfix) / relative rate: ~37%
@mbluj