Skip to content
Snippets Groups Projects

HLT1 UT-related tracking updates

Open Jiahui Zhuo requested to merge jzhuo_UTUpdatesMaster into master

Introduction

This MR includes five different sets of changes, each grouped into a separate Git commit. It is recommended to review the commits individually. :wink:

Needs lhcb-datapkg/ParamFiles!127

FYI: @adeoyang @bjashal @vsvintoz @vkholoim @dovombru @mveghel @cagapopo @gligorov (Feel free to tag additional people as needed :smile: )

0. Legacy code cleanup and new features

  • Remove unused dumpers (downstream track and downstream composite).
  • Remove the unused TrackMatchingConsts::MagnetParametrization.
  • Add a new helper, LinearAlgebra.cuh, for SolveCholesky<N>.
  • Add make_short2 in the CPU backend.

1. New UT Decoding

  • Change the UT hits container layout from x-based sector groups to y-based sector groups.
  • Implement new UT decoding, mapping each thread to a cluster.
  • Adapt matching and downstream tracking to the new UT hits layout.

Presented in WP2

New layout

image

Throughput improvement

image

2. New CompassUT

  • Implement a new CompassUT using the updated UT hits layout.

Presented in WP2

Performance improvement

image

3. New Downstream Tracking

  • Implement new Downstream tracking, replacing the (0,0) assumption with a general triplet search.

Presented in WP2

Performance improvement

image

4. Adaptive Thresholds for Hardcoded Limits

  • Add adaptive thresholds and proper counters in Matching.
  • Add adaptive thresholds and proper counters in Downstream.

The event will be reconstructed up to N times (default = 2): when it exceeds the hardcoded container limit, the search window size is scaled down by half, and the event is reprocessed.

Throughput in MEPs data

image

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
  • Arthur Marius Hennequin
  • Arthur Marius Hennequin
  • Arthur Marius Hennequin
  • mentioned in issue Moore#915 (closed)

  • Dorothea Vom Bruch requested review from @dtou

    requested review from @dtou

  • requested review from @ascarabo

  • Jiahui Zhuo added 6 commits

    added 6 commits

    • 8673510b - Legacy code cleanup and add more functionality
    • f9c1684c - Implement the new UT decoding
    • 610fd5d5 - Implement the new CompassUT
    • 6e77bac7 - Implement the new Downstream
    • 51b2f462 - Add adaptive threshold in hardcoded limits
    • 91bd583f - [REMOVE BEFORE MERGE] hack to fix CI/CD pipeline in GitLab

    Compare with previous version

  • Jiahui Zhuo added 4 commits

    added 4 commits

    • aa3106c6 - Implement the new CompassUT
    • 898aaaae - Implement the new Downstream
    • 886246eb - Add adaptive threshold in hardcoded limits
    • f0fec5d4 - [REMOVE BEFORE MERGE] hack to fix CI/CD pipeline in GitLab

    Compare with previous version

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