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
prevent PAT photon iso variables from overwriting Reco ones #8493
prevent PAT photon iso variables from overwriting Reco ones #8493
Conversation
A new Pull Request was created by @gzevi for CMSSW_7_5_X. prevent PAT photon iso variables from overwriting Reco ones It involves the following packages: DataFormats/PatCandidates @cmsbuild, @vadler, @nclopezo, @monttj can you please review it and eventually sign? Thanks. |
please test |
The tests are being triggered in jenkins. |
Note: when we switch to ROOT6 you'll be able to use the C++11 mechanism of saying you want to use a function for the base class:
|
-1 Tested at: bf7130f ---> test runtestPhysicsToolsPatAlgos had ERRORS you can see the results of the tests here: |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar |
+1 |
…h_reco4 prevent PAT photon iso variables from overwriting Reco ones
This is a different solution, suggested by @monttj and @lgray in #7761, to the issue of pat::Photon variables having the same name as reco::Photon variables.
Instead of renaming PAT variables (which would introduce an inconsistency between different PAT objects), we force PAT variables to return the corresponding Reco values.
This commit undoes the changes of #7764, which was not approved.
This commit is also backported to 74X in #8494