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

Phase 2 GTT Add Extended Jets to buffers for GlobalTrigger #1248

Open
wants to merge 1 commit into
base: phase2-l1t-integration-14_0_0_pre3
Choose a base branch
from

Conversation

NJManganelli
Copy link

PR description:

This PR adds the GTT Displaced TrackJets into the emulation buffers from GTT to GlobalTrigger

As with the PR to add displaced HT, this is not time-critical, but also does not affect workflows for the AR (only being called by GTT F/W Emulation buffer workflow, createFirmwareInputFiles_cfg.py)

PR validation:

scram b code-checks
scram b code-format
scram b
cmsRun createFirmwareInputFiles_cfg.py maxEvents=1000 format=APx inputFiles=... --> inserts Displaced TrackJet words into GTT buffers for OutputToGlobalTrigger

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

This will need a forward-port to master

@NJManganelli
Copy link
Author

If this gets approved, maintainers feel free to convert the PR to master from Draft (or I can do so whenever); no expected changes except ones needed here.

@aloeliger
Copy link

@NJManganelli I would take this straight to CMSSW. I am hopeful that we are going to add in a bunch of muon PRs and then close v9 and this round of phase 2 AR upgrades. I don't want this to just languish here, so just having it in central CMSSW will be the the quickest way to get it into the next AR IB (If one gets made, if I have my way we will make this whole fork obsolete in favor of just operating in central CMSSW).

@NJManganelli
Copy link
Author

Getting rid of IB(s) would be great! I'll convert the master PR to fully open

@triggerDoctor
Copy link

Hello, I'm triggerDoctor. @aloeliger is testing this script for L1T offline software validation.

Attempts to compile this PR succeeded!

Info Value
return code 0
command eval scramv1 runtime -sh && scram b -j 8

@triggerDoctor
Copy link

Hello, I'm triggerDoctor. @aloeliger is testing this script for L1T offline software validation.

I found no issues with the code checks!

Info Value
return code 0
command eval scramv1 runtime -sh && scram b -k -j 8 code-checks && scram b -k -j 8 code-checks

I found no issues with the headers!

Info Value
return code 0
command eval scramv1 runtime -sh && scram b -k -j 8 check-headers

@triggerDoctor
Copy link

Hello, I'm triggerDoctor. @aloeliger is testing this script for L1T offline software validation.

I found 2 files that did not meet formatting requirements:

  • L1Trigger/L1TMuon/src/MicroGMTConfiguration.cc
  • L1Trigger/Phase2L1GMT/plugins/Phase2L1TGMTFwdMuonTranslator.cc

Please run scram b code-format to auto-apply code formatting

Info Value
return code 0
command eval scramv1 runtime -sh && scram b -k -j 8 code-format-all

@triggerDoctor
Copy link

Hello, I'm triggerDoctor. @aloeliger is testing this script for L1T offline software validation.

This PR passes available unit tests!

Info Value
return code 0
command eval scramv1 runtime -sh && scram b runtests

@triggerDoctor
Copy link

Hello, I'm triggerDoctor. @aloeliger is testing this script for L1T offline software validation

I found a non-zero return code running the relval workflows for this PR!

Info Value
return code 1
command eval scramv1 runtime -sh && runTheMatrix.py --what upgrade -l 26834.78

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