-
Notifications
You must be signed in to change notification settings - Fork 10
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
Make option to do full matching+trigger efficiency plots #82
Comments
Is this something that needs development on the code or is this "just" config writing? Maybe something @LukasEbeling can do to gain familiarity with the code? |
Not really config writing: I was thinking we could remove the need for separate matching and trigger configs -> and just add an option to do the trigger plot too. Basically I would like to define 1 config e.g. Electron_Barrel, that would produce (selectable) three types of plots:
There is not need to have three configs imo. I think you have the better overview Daniel, so I let you decide how to implement that best. |
Just to understand what we're talking about:
ElectronsMatchingEndcap:
sample: DYLL_M50
version: V44nano
match_test_to_ref: True
reference_object:
object: "GenPart"
x_arg: "pt"
label: "Gen Electrons"
cuts:
event:
- "(({statusFlags}>>7)&1) == 1"
- "abs({pdgId}) == 11"
- "abs({eta}) > 1.5"
object:
- "abs({eta}) < 2.4"
test_objects:
L1EG:default: "pt"
L1tkElectron:NoIso: "pt"
# L1tkElectron:NoIsoNoLowPtID: "pt"
L1tkElectron:Iso: "pt"
xlabel: "Gen. $p_T$ (GeV)"
ylabel: "Matching Efficiency (Endcap)"
binning:
min: 0
max: 100
step: 3
ElectronsTriggerEndcap:
sample: DYLL_M50
version: V44nano
match_test_to_ref: False
reference_object:
object: "GenPart"
x_arg: "pt"
label: "Gen Electrons"
cuts:
event:
- "(({statusFlags}>>7)&1) == 1"
- "abs({pdgId}) == 11"
- "abs({eta}) > 1.5"
object:
- "abs({eta}) < 2.8"
test_objects:
L1EG:default:endcap: "pt"
L1tkElectron:NoIso:endcap: "pt"
# L1tkElectron:NoIsoNoLowPtID:endcap: "pt"
L1tkElectron:Iso:endcap: "pt"
thresholds: [10, 20, 30, 40]
scalings:
method: "naive"
threshold: 0.95
xlabel: "Gen. pT (GeV)"
ylabel: "Trigger Efficiency (endcap, L1 $p_T > <threshold>$ GeV)"
binning:
min: 0
max: 100
step: 1.5
ElectronsTriggerEndcap:
sample: DYLL_M50
version: V44nano
match_test_to_ref: True
reference_object:
object: "GenPart"
x_arg: "pt"
label: "Gen Electrons"
cuts:
event:
- "(({statusFlags}>>7)&1) == 1"
- "abs({pdgId}) == 11"
- "abs({eta}) > 1.5"
object:
- "abs({eta}) < 2.8"
test_objects:
L1EG:default:endcap: "pt"
L1tkElectron:NoIso:endcap: "pt"
# L1tkElectron:NoIsoNoLowPtID:endcap: "pt"
L1tkElectron:Iso:endcap: "pt"
thresholds: [10, 20, 30, 40]
scalings:
method: "naive"
threshold: 0.95
xlabel: "Gen. pT (GeV)"
ylabel: "Trigger Efficiency (endcap, L1 $p_T > <threshold>$ GeV)"
binning:
min: 0
max: 100
step: 1.5 Do you suggest leaving the plot definitions as they are and just putting them all in one file? In that case the feature would be to implement different command options that specify which plot you want. |
As this always comes up during the reviews, we need to add the possibility to produce "proper" trigger efficiency plots, which are essentially matching + trigger , i.e. the efficiency of matching a gen/ref object to a L1 object passing a certain threshold.
I suggest to produce these directly from the existing trigger plot configs, i.e. adding these plots to the trigger-only efficiency plots.
@danielhundhausen can you look into this?
The text was updated successfully, but these errors were encountered: