Skip to content
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

trigger matching #193

Merged
merged 28 commits into from
Sep 14, 2015
Merged

trigger matching #193

merged 28 commits into from
Sep 14, 2015

Conversation

jshlee
Copy link
Contributor

@jshlee jshlee commented Sep 14, 2015

added patTriggerObjectStandAlones for trigger matching
plus updates to h2mu analysis


public:
static float deltaR(float e1,float p1,float e2,float p2);
static float deltaR(reco::LeafCandidate & l1, reco::LeafCandidate & l2){return deltaR(l1.eta(),l1.phi(),l2.eta(),l2.phi());}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are standard deltaR function in the CMSSW. why don't you use it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep ok

@jshlee
Copy link
Contributor Author

jshlee commented Sep 14, 2015

by keeping only the electron and muon patTriggerObjectStandAlones we save a lot of space

patTriggerObjectStandAlones_selectedPatTrigger__PAT. 12642.8 1827.79
reduces to
patTriggerObjectStandAlones_catTrigger__CAT. 2535.65 355.238

@jhgoh
Copy link
Contributor

jhgoh commented Sep 14, 2015

good to see trigger info without restriction 👍

jhgoh added a commit that referenced this pull request Sep 14, 2015
This PR changes how to store trigger results and objects in CATTools. 
Dataformat is changed to use standard objects defined in PAT. Event size is reduced by selecting muon and electron triggers only - this is enough in quark analyses covered in Korean group.
Next production is expected with this new trigger info format.

There are other h2mumu analysis code changes in this PR which is independent of the central production.
@jhgoh jhgoh merged commit 9ed0fd4 into vallot:cat75x Sep 14, 2015
@jhgoh jhgoh mentioned this pull request Sep 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants