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

L1 HLT Studies #95

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

aloeliger
Copy link
Contributor

The Commit/Tools

This commit contains two parallel committed tools for the L1+HLT Acceptance studies. Previously in this repository were seeds.py and TriggerAcceptance.py, I have committed seeds_L1.py and triggerAcceptance_L1.py, designed to emulate the functionality of these two tools on L1 ntuples.

The scripts

  • seeds_L1.py: There are some functionality changes to this script versus seeds.py, the largest of them being the usage of regular expressions to do the finding of the L1HLT seeds in the HLT configuration, opposed to literal string matching. This shortens the code somewhat. It also takes a list/csv of L1 ntuples now, as opposed to nanoAOD. It also outputs the results in JSON instead of CSV for easier parsing.
  • triggerAcceptance_L1.py: This is as straight a port of TriggerAcceptance.py as is allowed by the changes to seeds.py (some of the histogram code is a literal copy and paste). The largest change involves how the L1 bit names are gotten. Since L1 ntuples do not carry the string name of the L1 bit, just the bit number (to my knowledge), a simple copy text copy of the oms L1 bit table is provided as input. The remaining changes are all just changes to work with the seeds_L1.py output.

Still To Do

  • A proper readme for the usage of the tools in this repository
  • It would be nice to not have to use an oms copied text file for bit names. There is an oms API available, but I did not use it for simplicity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant