Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 167
    • Merge requests 167
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • atlasatlas
  • athenaathena
  • Merge requests
  • !57629

Bugfixing C-side symmetry in the jFEX TOBs

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Sergi Rodriguez Bosca requested to merge serodrig/athena:Cside_correction into 22.0 Oct 18, 2022
  • Overview 11
  • Commits 2
  • Pipelines 2
  • Changes 5

This MR is meant to solve a bug in the local FPGA eta and phi. The C-side should be mirrored. The picture below shows one hemisphere for the globalEta (integer, using the local TOB eta) and the eta as a floating value Screenshot_from_2022-10-18_14-13-42

TOBs affected with this MR:

  • L1_jFexTauRoI
  • L1_jFexSRJetRoI
  • L1_jFexLRJetRoI

It is expected that this changes the trigger counts, since seeding is done with the globalEta coordinate.

Edited Oct 18, 2022 by Sergi Rodriguez Bosca
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: Cside_correction