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
Making dEtaIn seed available for ID purposes #6300
Conversation
A new Pull Request was created by @Sam-Harper for CMSSW_7_3_X. Making dEtaIn seed available for ID purposes It involves the following packages: DataFormats/EgammaCandidates @cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). This pull request will be automatically merged. |
Making dEtaIn seed available for ID purposes
This pull request simplifies the calculation of the new ID variable dEtaInSeed presented recently in the e/gamma meeting. As the name suggests, it is dEtaIn but computed using the eta of the seed cluster rather than the supercluster. It is simply dEtaIn-sc eta + seed eta. I have added a function to GsfElectron which makes it, it would be nice if we could have this, it just simplifies things and avoids mistakes but we can live without.
I have also added a class to the ID framework which can cut on this variable.