Skip to content

[PWGDQ] make qaMatching AO2D index merger-safe#16129

Merged
mcoquet642 merged 1 commit intoAliceO2Group:masterfrom
nivram-phy:fix-qamatching-hyperloop-index
May 6, 2026
Merged

[PWGDQ] make qaMatching AO2D index merger-safe#16129
mcoquet642 merged 1 commit intoAliceO2Group:masterfrom
nivram-phy:fix-qamatching-hyperloop-index

Conversation

@nivram-phy
Copy link
Copy Markdown
Contributor

@nivram-phy nivram-phy commented May 6, 2026

Summary

  • make the qaMatching event-reference column use a merger-safe custom index label
  • preserve canonical AO2D tree naming for the qaMatching output tables

Problem

The previous qaMatching AO2D output produced an event-reference branch that was not resolved correctly by o2-aod-merger and caused downstream Hyperloop issues.

@github-actions github-actions Bot added the pwgdq label May 6, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

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

@github-actions github-actions Bot changed the title PWGDQ: make qaMatching AO2D index merger-safe [PWGDQ] make qaMatching AO2D index merger-safe May 6, 2026
@nivram-phy nivram-phy marked this pull request as ready for review May 6, 2026 14:18
@nivram-phy
Copy link
Copy Markdown
Contributor Author

@aferrero2707 This is the new PR with the unbounded index issue resolved (hopefully). I chcked it locally with the o2-aod-merger

@mcoquet642 mcoquet642 merged commit e743331 into AliceO2Group:master May 6, 2026
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants