search

KIT-CMS / Excalibur

Analysis repository for Z+Jet studies
1 stars 4 forks source link

LeptonRecoSFProducer #77

Closed cverstege closed 2 years ago

cverstege commented 2 years ago

Außerdem könntest du vielleicht die Config-Änderungen von der Code-Implementierung entkoppeln. Das interessiert in Zukunft vermutlich sowieso niemanden mehr, aber es wäre ein wenig übersichtlicher.

Sind zwei getrennte commits. Man kann auch erstmal nur den einen Cherry Picken. Ich kann den Branch ja beliebig rebasen.

cverstege commented 2 years ago

I think this change can be disregarded as the recommendation now is to assume 1 for the tracking efficiency and I don't need it anymore. https://twiki.cern.ch/twiki/bin/viewauth/CMS/MuonUL2016#Tracking_efficiency https://twiki.cern.ch/twiki/bin/viewauth/CMS/MuonUL2017#Tracking_efficiency https://twiki.cern.ch/twiki/bin/viewauth/CMS/MuonUL2018#Tracking_efficiency

cverstege commented 2 years ago

Sorry, I might still need this work for the RECO efficiencies https://twiki.cern.ch/twiki/bin/viewauth/CMS/MuonUL2018#RECO_efficiency

Reopening, but still WIP

cverstege commented 2 years ago

Ich hoffe dass c2500172c14e1423a837dde6ae11e5e15aad2a86 nichts kaputt macht, allerdings sah die Zusammensetzung für den gemeinsamem SF komisch aus. Ich denke dass eine einfach Mutliplikation hier richtig ist. So wird es auch bei allen anderen SF gemacht. Ansonsten ist der neue RecoSF producer bereit zum review, siehe https://twiki.cern.ch/twiki/bin/viewauth/CMS/MuonUL2018#RECO_efficiency

cverstege commented 2 years ago

Should be ready now.

cverstege commented 2 years ago

Please don't merge yet. I'm evaluating if I maybe need to also use Triggers for high pt.

RHofsaess commented 2 years ago

I would like to merge this and move it to another PR to avoid merge conflicts.

RHofsaess commented 2 years ago

Branch can be rebased and reused later.