Skip to content

[Common] Adding muonQA task #11438

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
iarsene merged 1 commit into from
Jun 4, 2025
Merged

[Common] Adding muonQA task #11438

iarsene merged 1 commit into from
Jun 4, 2025

Conversation

@ppoava
Copy link
Contributor

@ppoava ppoava commented Jun 2, 2025

Adding muon QA task for tracks in MCH-MID and MFT-MCH-MID. Can be used for MCH and MFT-MCH relative alignment studies.

@github-actions github-actions bot added the common label Jun 2, 2025
@github-actions
Copy link

github-actions bot commented Jun 2, 2025 •
edited
Loading

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

@ppoava ppoava marked this pull request as ready for review June 2, 2025 13:39
@ppoava ppoava requested review from a team, alibuild, ddobrigk, iarsene, jgrosseo and ktf as code owners June 2, 2025 13:39
iarsene
iarsene previously approved these changes Jun 2, 2025
View reviewed changes
@iarsene iarsene enabled auto-merge (squash) June 2, 2025 13:41
@alibuild
Copy link
Collaborator

alibuild commented Jun 2, 2025

Error while checking build/O2Physics/o2 for aaccaa8 at 2025-06-02 16:06:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/11438-slc9_x86-64/0/Common/Tasks/qaMuon.cxx:2027:14: error: variable 'mfttrack' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/11438-slc9_x86-64/0/Common/Tasks/qaMuon.cxx:1783:56: error: unused parameter 'bcs' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/11438-slc9_x86-64/0/Common/Tasks/qaMuon.cxx:1783:75: error: unused parameter 'muons' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/11438-slc9_x86-64/0/Common/Tasks/qaMuon.cxx:1862:61: error: unused parameter 'bcs' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/11438-slc9_x86-64/0/Common/Tasks/qaMuon.cxx:1783:56: error: unused parameter 'bcs' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/11438-slc9_x86-64/0/Common/Tasks/qaMuon.cxx:1783:75: error: unused parameter 'muons' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

auto-merge was automatically disabled June 3, 2025 06:57

Head branch was pushed to by a user without write access

iarsene
iarsene previously approved these changes Jun 3, 2025
View reviewed changes
@iarsene iarsene enabled auto-merge (squash) June 3, 2025 07:47
auto-merge was automatically disabled June 3, 2025 10:13

Head branch was pushed to by a user without write access

@ppoava
[Common] Adding muonQA task
54beb48 
Adding muon QA task for tracks in MCH-MID and MFT-MCH-MID. Can be used for MCH and MFT-MCH relative alignment studies.
@ppoava ppoava force-pushed the dev_muonQA_task_rebased branch from 60abe7a to 54beb48 Compare June 3, 2025 10:17
@iarsene iarsene enabled auto-merge (squash) June 3, 2025 11:03
@iarsene iarsene merged commit 08d95c0 into AliceO2Group:master Jun 4, 2025
11 of 14 checks passed
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
@ppoava @jinhyunni
[Common] Adding muonQA task (AliceO2Group#11438)
a283a29
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Jun 11, 2025
@ppoava
[Common] Adding muonQA task (AliceO2Group#11438)
66031df
EmilGorm pushed a commit to EmilGorm/O2Physics that referenced this pull request Jun 12, 2025
@ppoava @EmilGorm
[Common] Adding muonQA task (AliceO2Group#11438)
b3968cb
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 2025
@ppoava
[Common] Adding muonQA task (AliceO2Group#11438)
606f126
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
@ppoava
[Common] Adding muonQA task (AliceO2Group#11438)
9a37d71
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
@ppoava @smaff92
[Common] Adding muonQA task (AliceO2Group#11438)
38447fb
@ppoava ppoava deleted the dev_muonQA_task_rebased branch July 16, 2025 07:00
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jul 16, 2025
@ppoava @romainschotter
[Common] Adding muonQA task (AliceO2Group#11438)
7247ce4
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
@ppoava @CesarOmarRA
[Common] Adding muonQA task (AliceO2Group#11438)
89d47df
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
@ppoava @jloemker
[Common] Adding muonQA task (AliceO2Group#11438)
2e4049a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iarsene iarsene iarsene approved these changes

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

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

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

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

Assignees

No one assigned

Labels

common

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ppoava @alibuild @iarsene