Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • L LHCb
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 193
    • Issues 193
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Jira
    • Jira
  • Merge requests 42
    • Merge requests 42
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • LHCb
  • Merge requests
  • !3735

Check SciFi banks and sort them by hand

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Louis Henry requested to merge lohenry-checkBanks into master Aug 18, 2022
  • Overview 16
  • Commits 12
  • Pipelines 10
  • Changes 4

This will slow down the decoding quite a bit but we check for scenarios that should never happen and yet do in the data.

  • if a bank is sent with no header (would create a segfault);
  • if a bank does not have an even size;
  • if a bank has an absurd number of clusters (> number of links * maximum number of clusters per link);
  • if a link is larger than 24 (would create a segfault);
  • if a link sent more than 24 clusters.

Additionally, we sort clusters directly. Unsorted clusters should only show up in station 3 but seem to rarely occur somewhere else. An issue will be created.

We keep small_vectors rather than static_vectors to avoid a segfault in case a link sent more than 24 clusters.

Each of these counters now comes with an histogram which helps locate the bank or unique link that was responsible.

Edited Aug 19, 2022 by Louis Henry
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: lohenry-checkBanks