Skip to content
Snippets Groups Projects

Automatic derivation of sensor lookup table in UT

Merged Michel De Cian requested to merge decianm-SectorSearchUT into master
  • replace hard-coded lookup table with derivation
  • bring back sensor-swaps for old databases (addresses: #305 (closed))
  • simplify a few things

Merge request reports

Merge request pipeline #5822594 passed

Merge request pipeline passed for d9e1c736

Approval is optional

Merged by Rosen MatevRosen Matev 1 year ago (Jul 10, 2023 9:43pm UTC)

Merge details

Pipeline #5873999 passed

Pipeline passed for 8a2b722b on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Michel De Cian added 1 commit

    added 1 commit

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    Compare with previous version

  • Michel De Cian added 2 commits

    added 2 commits

    • e8baf4d9 - replace hardcoded number with derived number from UTInfo::DetectorNumbers
    • 390ec4a7 - reverse logic of 'swapped'

    Compare with previous version

  • Michel De Cian added 20 commits

    added 20 commits

    • 390ec4a7...fac15526 - 10 commits from branch master
    • 24c0439b - predefine and move some numbers related to UT detector (readout) to UTInfo,...
    • cdde7bc9 - rename constants to be more descriptive
    • 7ab193c7 - more cleanup of unused code, some simplifications
    • cc1b69e2 - simplify and make more clear
    • c7c4a3d6 - replace hard-coded lookup table with derivation, bring back sensor-swaps for...
    • f3cf7fa8 - remove const
    • a089cdc1 - Fixed formatting
    • a6e627c9 - replace hardcoded number with derived number from UTInfo::DetectorNumbers
    • 856b2e16 - reverse logic of 'swapped'
    • 089fa3fd - format

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    • 542cbea9 - add static sanity check, name a magic number

    Compare with previous version

  • assigned to @mzdybal

  • Michel De Cian marked this merge request as ready

    marked this merge request as ready

  • Edited by Software for LHCb
  • Michel De Cian added 1 commit

    added 1 commit

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    • 0a2f07fc - declare and define variables together

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading