Skip to content

[PWGUD] MC analysis for flow#15626

Open
miedema-11 wants to merge 6 commits intoAliceO2Group:masterfrom
miedema-11:master
Open

[PWGUD] MC analysis for flow#15626
miedema-11 wants to merge 6 commits intoAliceO2Group:masterfrom
miedema-11:master

Conversation

@miedema-11
Copy link
Copy Markdown
Contributor

@miedema-11 miedema-11 commented Apr 2, 2026

No description provided.

@github-actions github-actions bot added the pwgud label Apr 2, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026 •
edited
Loading

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

@github-actions github-actions bot changed the title MC analysis for flow [PWGUD] MC analysis for flow Apr 2, 2026
@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented Apr 2, 2026

Error while checking build/O2Physics/o2 for fc7b1e9 at 2026-04-02 15:31:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGUD/Tasks/flowMcUpc.cxx:131:16: error: unused variable 'phi' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGUD/Tasks/flowMcUpc.cxx:171:14: error: unused variable 'phi' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGUD/Tasks/flowMcUpc.cxx:95:33: error: unused parameter 'collision' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@alibuild alibuild mentioned this pull request Apr 2, 2026
Closed
@miedema-11 miedema-11 marked this pull request as draft April 2, 2026 14:23
@miedema-11 miedema-11 marked this pull request as ready for review April 2, 2026 14:37
@miedema-11 miedema-11 marked this pull request as draft April 2, 2026 15:36
@miedema-11
Comment out phi calculation in flowMcUpc.cxx
f179a75 
Comment out the calculation of phi for mcParticles and tracks.
@miedema-11 miedema-11 marked this pull request as ready for review April 2, 2026 16:41
@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented Apr 2, 2026

Error while checking build/O2Physics/o2 for aad8ae2 at 2026-04-02 21:12:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGUD/Tasks/flowMcUpc.cxx:129:16: error: unused variable 'phi' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGUD/Tasks/flowMcUpc.cxx:169:14: error: unused variable 'phi' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@miedema-11
Copy link
Copy Markdown
Contributor Author

miedema-11 commented Apr 3, 2026

sorry for this error, but I am not sure what to do about the CMakeList.cxx

@miedema-11
Copy link
Copy Markdown
Contributor Author

miedema-11 commented Apr 3, 2026

If it is caused by my code, please let me know. Thank you so much

@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented Apr 3, 2026

Error while checking build/O2Physics/o2 for f179a75 at 2026-04-03 19:36:

No log files found

Full log here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

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

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

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

pwgud

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@miedema-11 @alibuild