Skip to content

Conversation

@mhemmer-cern
Copy link
Collaborator

…acks` table

  • Add DeltaEta and DeltaPhi values to EMCALMatchedTracks table
  • Add PWGJE/Core/utilsTrackMatchingEMC.h where we do simple track matching. Previously we used the jetutils which matched tracks to clusters and clusters to track both. To save time we now use our own utils header and do only one. Also to make the code more easily understandble added struct MatchResult instead of using a complex tuple of vector abomination.
  • Fix O2linter warnings and includes for the touched .cxx files

…acks` table

- Add DeltaEta and DeltaPhi values to `EMCALMatchedTracks` table
- Add PWGJE/Core/utilsTrackMatchingEMC.h where we do simple track matching. Previously we used the jetutils which matched tracks to clusters and clusters to track both. To save time we now use our own utils header and do only one. Also to make the code more easily understandble added struct `MatchResult` instead of using a complex tuple of vector<vector> abomination.
- Fix PWGJE/Tasks/emcTmMonitor.cxx O2linter warnings and includes
@github-actions
Copy link

O2 linter results: ❌ 9 errors, ⚠️ 0 warnings, 🔕 19 disabled

@mhemmer-cern mhemmer-cern enabled auto-merge (squash) July 25, 2025 12:14
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @mhemmer-cern.

@jokonig
Copy link
Contributor

jokonig commented Jul 25, 2025

Looks good to me

@mhemmer-cern mhemmer-cern merged commit 8c07c8d into AliceO2Group:master Jul 25, 2025
12 of 13 checks passed
@mhemmer-cern mhemmer-cern deleted the EMCAL-689-TM branch July 28, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants