Skip to content
Snippets Groups Projects

Add protections to muon decoding

Merged Saverio Mariani requested to merge samarian_protect_muon_decoding into master

MR following up LHCb!3736 (merged) and adding protections to the Allen muon decoding to avoid the crashes that were reported in LHCb#242 because of an inadequate raw bank size. As resulting from some offline discussions, the full raw bank has to be skipped in case a Tell40 link gives corrupted info.

Performed tests:

  • Allen launched from Moore can decode the full data file indicated in LHCb#242 with no crashes
  • Moore hlt1_hlt2_muon_hits, comparing Rec and Allen decoded hits, is successful
  • throughput on data of the muon_decoding_sequence slows down by a relative 5.9%. Considering the muon decoding only represents a 0.8% of the overall sequence, this is expected to be acceptable
Edited by Saverio Mariani

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading