Skip to content

[PWGDQ] refactoring of some matching QA member functions#16704

Merged
mcoquet642 merged 1 commit into
AliceO2Group:masterfrom
aferrero2707:PWGDQ-matching-qa-functions-refactoring
Jun 17, 2026
Merged

[PWGDQ] refactoring of some matching QA member functions#16704
mcoquet642 merged 1 commit into
AliceO2Group:masterfrom
aferrero2707:PWGDQ-matching-qa-functions-refactoring

Conversation

@aferrero2707

Copy link
Copy Markdown
Contributor

The refactoring removes some unneeded retrieval of track references from the input tables.

@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown

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

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 051f3a1 at 2026-06-17 11:15:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGDQ/Tasks/qaMatching.cxx:2658:73: error: 'muonTrack1' was not declared in this scope; did you mean 'muonTracks'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGDQ/Tasks/qaMatching.cxx:2659:73: error: 'muonTrack2' was not declared in this scope; did you mean 'muonTracks'?
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 051f3a1 at 2026-06-17 11:20:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGDQ/Tasks/qaMatching.cxx:2658:73: error: 'muonTrack1' was not declared in this scope; did you mean 'muonTracks'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGDQ/Tasks/qaMatching.cxx:2659:73: error: 'muonTrack2' was not declared in this scope; did you mean 'muonTracks'?
ninja: build stopped: subcommand failed.

Full log here.

@aferrero2707 aferrero2707 marked this pull request as draft June 17, 2026 09:35
The refactoring removes some unneeded retrieval of track references
from the input tables.
@aferrero2707 aferrero2707 force-pushed the PWGDQ-matching-qa-functions-refactoring branch from 051f3a1 to 87ab630 Compare June 17, 2026 09:49
@aferrero2707

Copy link
Copy Markdown
Contributor Author

The build failure was due to the parallel merging of another PR, I have rebased the code once more now that no additional PRs are pending.

@aferrero2707 aferrero2707 marked this pull request as ready for review June 17, 2026 09:55
@mcoquet642 mcoquet642 merged commit e869788 into AliceO2Group:master Jun 17, 2026
13 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.

3 participants