Skip to content

[PWGUD] Adding correlation study #11333

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
abylinkin merged 38 commits into from
May 30, 2025
Merged

[PWGUD] Adding correlation study #11333

abylinkin merged 38 commits into from
May 30, 2025

Conversation

@JosueMtzGar
Copy link
Contributor

@JosueMtzGar JosueMtzGar commented May 23, 2025

No description provided.

JosueMtzGar added 28 commits May 9, 2024 14:59
@JosueMtzGar
improving the analysis
fc80f61
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
25763be
@JosueMtzGar
improving the analysis
4cae5d6
@JosueMtzGar
new changes
7f09d3d
@JosueMtzGar
new changes
d854bdd
@JosueMtzGar
Merge remote-tracking branch 'upstream/master' into localtestPWGUD
b33a3b6
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
4ad5488
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
ecaad17
@JosueMtzGar
adding angular correlation analysis
7234a79
@JosueMtzGar
saving the last changes
2768e38
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
63d8c61
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
b18cd8e
@JosueMtzGar
adding helper to pair cuts for UD tables
03cd5be
@JosueMtzGar
new changes
62739d2
@JosueMtzGar
new changes, using correlation container. output object empty
907fc67
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
3101f1c
@JosueMtzGar
probe of correlation container tutorial
c463936
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
3e45828
@JosueMtzGar
a little change to test
ddd1083
@JosueMtzGar
recent changes on the correlation analysis, container still empty
81251f2
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
d257ef9
@JosueMtzGar
saving updates to change to Lumi work
35e7ea8
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
4768865
@JosueMtzGar
new changes
48cd940
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
ad7514f
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
0154bcf
@JosueMtzGar
new changes to do correlations
1dff562
@JosueMtzGar
Merge branch 'master' into localtestPWGUD
7114730
@alibuild
Copy link
Collaborator

alibuild commented May 23, 2025 •
edited
Loading

Error while checking build/O2Physics/o2 for 0ae71ee at 2025-05-24 06:59:

## sw/BUILD/O2Physics-latest/log
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
/sw/SOURCES/O2Physics/11333-slc9_x86-64/0/PWGUD/Tasks/upcPhotonuclearAnalysisJMG.cxx:541:100: error: unused parameter 'reconstructedTracks' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/11333-slc9_x86-64/0/PWGUD/Tasks/upcPhotonuclearAnalysisJMG.cxx:350:51: error: unused parameter 'collision' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/11333-slc9_x86-64/0/PWGUD/Tasks/upcPhotonuclearAnalysisJMG.cxx:341:28: error: unused parameter 'collision' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/11333-slc9_x86-64/0/PWGUD/Tasks/upcPhotonuclearAnalysisJMG.cxx:341:45: error: unused parameter 'multiplicity' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@JosueMtzGar JosueMtzGar marked this pull request as draft May 26, 2025 20:12
@JosueMtzGar JosueMtzGar marked this pull request as ready for review May 27, 2025 05:28
@abylinkin
Copy link
Collaborator

abylinkin commented May 27, 2025

I think O2linter errors like hard-coded masses should be fixed before mergin

@JosueMtzGar JosueMtzGar marked this pull request as draft May 28, 2025 23:16
@JosueMtzGar JosueMtzGar marked this pull request as ready for review May 29, 2025 06:30
@JosueMtzGar
Copy link
Contributor Author

JosueMtzGar commented May 29, 2025

Hi @abylinkin, I have resolved the most possible issues.

@abylinkin abylinkin enabled auto-merge (squash) May 29, 2025 07:06
@abylinkin abylinkin merged commit 0daa29d into AliceO2Group:master May 30, 2025
12 of 13 checks passed
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
@JosueMtzGar @jinhyunni
[PWGUD] Adding correlation study (AliceO2Group#11333)
e56ec25
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Jun 11, 2025
@JosueMtzGar
[PWGUD] Adding correlation study (AliceO2Group#11333)
1d5a87d
EmilGorm pushed a commit to EmilGorm/O2Physics that referenced this pull request Jun 12, 2025
@JosueMtzGar @EmilGorm
[PWGUD] Adding correlation study (AliceO2Group#11333)
59c5bb6
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 2025
@JosueMtzGar
[PWGUD] Adding correlation study (AliceO2Group#11333)
14c2141
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
@JosueMtzGar
[PWGUD] Adding correlation study (AliceO2Group#11333)
26969cc
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
@JosueMtzGar @smaff92
[PWGUD] Adding correlation study (AliceO2Group#11333)
0f02957
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jul 16, 2025
@JosueMtzGar @romainschotter
[PWGUD] Adding correlation study (AliceO2Group#11333)
d949390
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
@JosueMtzGar @CesarOmarRA
[PWGUD] Adding correlation study (AliceO2Group#11333)
2924e24
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
@JosueMtzGar @jloemker
[PWGUD] Adding correlation study (AliceO2Group#11333)
495a460
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abylinkin abylinkin abylinkin approved these changes

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@pbuehler pbuehler Awaiting requested review from pbuehler

@rolavick rolavick Awaiting requested review from rolavick rolavick is a code owner

Assignees

No one assigned

Labels

pwgud

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JosueMtzGar @alibuild @abylinkin