Skip to content
Snippets Groups Projects

New sourceID encoding and new UT ReadoutMap

Merged Hangyi Wu requested to merge ut-subsector-in-readoutmap into 2024-patches

Work with Detector!470 (merged), Rec!3880 (merged) and lhcb-conddb/lhcb-conditions-database!118 (merged)

  • Work with ReadoutMap with additional subsector bit
  • Provide sourceID encoding identical to real data (previously differ from data)
  • Fixed RawBank ordering by changing it from sorting-by-boardID to sorting-by-sourceID. This should match exactly how real data is organized.
  • Provide helpers to convert ChannelID to ECS name
Edited by Hangyi Wu

Merge request reports

Merge request pipeline #7193777 passed

Merge request pipeline passed for ce1f35a0

Approved by

Merged by Christopher Rob JonesChristopher Rob Jones 10 months ago (Apr 23, 2024 9:39am UTC)

Merge details

  • Changes merged into 2024-patches with d3452b60 (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Sebastien Ponce approved this merge request

    approved this merge request

    • Resolved by Hangyi Wu

      Does this MR also work with DetDesc? or will the decoding for MC with this MR be incompatible with the sourceID encoding with Boole DetDesc and something need to be done for Boole or in SIMCOND?

      Edited by Gloria Corti
  • Hangyi Wu added 1 commit

    added 1 commit

    • 1163539b - fixup! update DAQHelper and ReadoutTool for new map

    Compare with previous version

  • Hangyi Wu marked this merge request as draft from 1163539b

    marked this merge request as draft from 1163539b

  • Hangyi Wu marked this merge request as ready

    marked this merge request as ready

  • mentioned in issue Moore#699 (closed)

  • mentioned in issue Moore#700 (closed)

  • Edited by Software for LHCb
  • Hangyi Wu 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