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

Draft of muon showers implementation #62

Merged
merged 8 commits into from
Nov 2, 2021
Merged

Draft of muon showers implementation #62

merged 8 commits into from
Nov 2, 2021

Conversation

elfontan
Copy link
Collaborator

No description provided.

@elfontan elfontan marked this pull request as ready for review October 28, 2021 17:34
@elfontan
Copy link
Collaborator Author

@arnobaer @jheikkil:
I finally managed to test the updates and they are correctly working.
Here the steps to test the updates within a customised CMSSW area created to include Sven's changes to the L1TNtuples:

CMSSW SETUP:
cmsrel CMSSW_12_0_2
cd CMSSW_12_0_2/src
cmsenv
git cms-init
git-cms-addpkg L1Trigger/L1TGlobal
git-cms-addpkg L1Trigger/L1TNtuples
git-cms-addpkg DataFormats/L1TGlobal
git-cms-addpkg DataFormats/L1Trigger
git-cms-addpkg DataFormats/L1TMuon
git remote -v
git remote add cmssw_fromSven https://github.com/dildick/cmssw
git remote -v
git fetch cmssw_fromSven
git merge cmssw_fromSven/l1t-integration-CMSSW_12_0_2-uGT-hadronic-shower-trigger
scram b -j 8

MENUTOOLS SETUP (NOTE: logout and then go back to CMSSW_12_0_2/src; no cmsenv):
git clone --depth 1 https://github.com/cms-l1-dpg/L1MenuTools.git
cd L1MenuTools/rate-estimation
==> Three new files from this PR included!
wget https://raw.githubusercontent.com/dildick/L1Tutorials/draft-shower-menu-for-run3/tutorials/tme-seed-development/input/L1Menu_Collisions2022_v0_1_1_MuShowers.xml
bash configure.sh L1Menu_Collisions2022_v0_1_1_MuShowers.xml
cmsenv
mkdir -p objs/include
make -j 8

The PR can be reviewed and merged!
Thanks a lot Bernhard for your precious help.

@dildick
Copy link
Contributor

dildick commented Oct 28, 2021

I'll need to update the draft menu, because tight mapping changed. Basically I would change it to (drop the two loose trigger)

  <algorithm>
    <name>L1_MuShower_OneNominal</name>
    <expression>MUS0</expression>
    <index>34</index>
    <module_id>0</module_id>
    <module_index>34</module_index>
    <object_requirement>
      <name>MUS0</name>
      <type>MUS0</type>
      <comparison_operator>.ge.</comparison_operator>
      <threshold>+0.0000000000000000E+00</threshold>
      <bx_offset>0</bx_offset>
    </object_requirement>
  </algorithm>
  <algorithm>
    <name>L1_MuShower_OneTight</name>
    <expression>MUS1</expression>
    <index>35</index>
    <module_id>0</module_id>
    <module_index>35</module_index>
    <object_requirement>
      <name>MUS1</name>
      <type>MUS1</type>
      <comparison_operator>.ge.</comparison_operator>
      <threshold>+0.0000000000000000E+00</threshold>
      <bx_offset>0</bx_offset>
    </object_requirement>
  </algorithm>

so that nominal is mapped to MUS0 and tight is mapped to MUS1

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.

3 participants