Skip to content

[PWGDQ] Add SS/OS functionality to J/Psi-muon correlations task + fixes - #17289

Merged
mcoquet642 merged 8 commits into
AliceO2Group:masterfrom
kaareendrup:JPsiMuonCorrelationsFixes
Aug 3, 2026
Merged

[PWGDQ] Add SS/OS functionality to J/Psi-muon correlations task + fixes#17289
mcoquet642 merged 8 commits into
AliceO2Group:masterfrom
kaareendrup:JPsiMuonCorrelationsFixes

Conversation

@kaareendrup

Copy link
Copy Markdown
Contributor

This PR will add minor changes to J/Psi-muon correlations task:

  • Add extraction of SS dilepton pairs and save to different subdirectories
  • Remove manual dilepton cuts and use aod filters instead
  • Fix default value of ccdb url

@github-actions github-actions Bot added the pwgdq label Aug 2, 2026
@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Add SS/OS functionality to J/Psi-muon correlations task + fixes [PWGDQ] Add SS/OS functionality to J/Psi-muon correlations task + fixes Aug 2, 2026
@kaareendrup
kaareendrup marked this pull request as ready for review August 3, 2026 09:01
@alibuild

alibuild commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 00ff06a at 2026-08-03 11:18:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 1 errors and 0 warnings.'
Found 1 errors and 0 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/b08793249e4f1f25d53df56645bff9701f53904c/O2Physics-code-check/errors.txt
PWGDQ/Tasks/jPsiMuonCorrelations.cxx:143:20: error: no header providing "Form" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

Please consider the following formatting changes to AliceO2Group#17289
@mcoquet642
mcoquet642 enabled auto-merge (squash) August 3, 2026 11:44
@mcoquet642
mcoquet642 merged commit e0cd446 into AliceO2Group:master Aug 3, 2026
15 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