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 45
    • Merge requests 45
  • 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
  • !3738

Draft: FT: allow again for large clusters to be decoded

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Louis Henry requested to merge tmp-lohenry-largeClusters into master Aug 22, 2022
  • Overview 12
  • Commits 13
  • Pipelines 3
  • Changes 2

Large clusters in FT are supposed to be transmitted by a pair of clusters, with (size,frac) flags set to (1,1) and (1,0) for the beginning (end), respectively.

However in the data, they always start at (1,0) and end either with (1,0) or (1,1). This behaviour was previously taken as corrupt, and the cluster ignored. However, this arises from a possible miscommunication between the Tell40 and software teams, and will be solved. The clusters are real.

This MR, temporary as it breaks the intended large cluster treatment, allows to still form large clusters.

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