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
Bug fix in HLTJetCollectionsVBFFilter #7457
Bug fix in HLTJetCollectionsVBFFilter #7457
Conversation
Fix a bug where the code is using the index into the RefVector where the index into the collection is expected. This will result in Ref's to the wrong element being created.
A new Pull Request was created by @wddgit (W. David Dagenhart) for CMSSW_7_4_X. Bug fix in HLTJetCollectionsVBFFilter It involves the following packages: HLTrigger/JetMET @Martin-Grunewald, @perrotta, @cmsbuild, @nclopezo, @fwyzard can you please review it and eventually sign? Thanks. |
Note I am not familiar with this code. It is possible The Ref constructor used by this code will be deleted To illustrate the problem more clearly, the RefVector |
Thanks for catching this! |
please test |
The tests are being triggered in jenkins. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request will be automatically merged. |
Bug fix in HLTJetCollectionsVBFFilter
Fix a bug where the code is using the index
into the RefVector where the index into the
collection is expected. This will result in
Ref's to the wrong element being created.