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

Inclusion of the displaced jet bit #74

Merged
merged 4 commits into from Feb 10, 2022
Merged

Inclusion of the displaced jet bit #74

merged 4 commits into from Feb 10, 2022

Conversation

elfontan
Copy link
Collaborator

@elfontan elfontan commented Jan 27, 2022

Preliminary implementation of the displaced jet bit as an info for jet objects coming from the Calo Layer 2.
We have a 1-bit LUT where the value can be only 0-1.
Reference JIRA Ticket: CMSLITDPG-963.

Open items:

  1. Name of the variable in the L1AnalysisL1UpgradeDataFormat --> final implementation in the emulator not available yet
  2. Do we need to read a specific LUT like for the Iso or IP?

@elfontan elfontan changed the title Preliminary change to include the displaced jet bit Inclusion of the displaced jet bit Jan 27, 2022
@elfontan
Copy link
Collaborator Author

elfontan commented Jan 29, 2022

The displaced jet bit hasn't officially been added to L1AnalysisL1UpgradeDataFormat already.
The full L1 emulator (Calo L1, L2, jet algo) by Gillian, along with the modifications to save the jet HwQual bit in the L1TNtuple, are in this commit.

In the JetBxCollection, the jet quality word (hwQual) is accessed with a jet iterator, it->hwQual(). The first bit of jet hwQual (bit 27) as a 1 bit LUT is used: 0 if the jet is not found to be delayed, and 1 if it is (meaning 2 LLP-flagged trigger towers are contained within it).

We have two reference PRs:

  • PR#36919 for the L1 and L2 Calo emulator, mostly relevent for our implementation;
  • PR#36858 for the uGT emulator.

Fix of the check on the LSB of the jet.hwQual changing the shift operator to AND operator.
@elfontan elfontan merged commit 1b31797 into master Feb 10, 2022
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.

None yet

1 participant