Closed GaelTouquet closed 6 years ago
Hi, no problem I can deal with that on Monday when I can get my hands on a computer.
Le 21 août 2018 15:03:49 GMT+02:00, Giovanni Petrucciani notifications@github.com a écrit :
gpetruc commented on this pull request.
- (1.479, 0.1562),
- (2.000, 0.1032),
- (2.200, 0.0859),
- (2.300, 0.1116),
- (2.400, 0.1321),
- (2.500, 0.1654)
- ] +)
+def get_effective_area(eta, area):
- eta = abs(eta)
- ea_defs = effective_areas[area]
- for etamax, ea in ea_defs:
- if eta < etamax:
- return ea
- return ea_defs[-1][1]
also, I think that this and the above should be hidden from
import *
by having names starting with_
, since they're not part of the public interface of the electron object and the name is unspecific enough that may clash with something else.-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/CERN-PH-CMG/cmg-cmssw/pull/738#pullrequestreview-148047717
Colin -- sent from my phone
@gpetruc I fully agree with your comments (and I had missed the LeptonAnalyzer because I do not use it). I am not completely sure how to proceed, so let me explain my point of view:
That being said, again, you're completely right, we need a clean way to deal with eras.
What I propose is to introduce a global configuration module for heppy (that's what I did on the FCC side for other reasons: https://github.com/cbernet/heppy/blob/master/configuration.py Based on this configuration, decisions are taken, e.g. here: https://github.com/cbernet/heppy/blob/master/utils/deltar.py#L14
We could do the same in CMS. The user could then specify the era in the config file like this: https://github.com/cbernet/fcc-ee-higgs/blob/master/analysis_ee_ZH_lltautau.py#L48
If you agree with this way to proceed, I will:
Please let me know what you think.
Hi, My proposal would be to take the EA setting from: (1) named function argument of the isolation function, (2) datamember of the electron object (3) data member of the electron class, and raise exception if neither of the 3 is available. The lepton analyzer can set it on the objects and then rely on (2)
perfect, working on it.
can be closed, overwritten by PR #741
To implement 2017 EA electron isolation properly, we realized that it was necessary to clean up the lepton isolation interface. Slight changes in client code will be necessary this time but we think that the new interface will be able to accommodate easily future implementations of the isolation.
The isolation has been perfectly synchronized with other groups in the h to tau tau analysis.