Skip to content
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

[Common] QA developments, cent work for more estimators #8999

Merged
merged 5 commits into from
Dec 21, 2024
Merged

[Common] QA developments, cent work for more estimators #8999

merged 5 commits into from
Dec 21, 2024

Conversation

ddobrigk
Copy link
Collaborator

No description provided.

@ddobrigk ddobrigk requested review from alibuild, jgrosseo, iarsene, ktf and a team as code owners December 14, 2024 20:01
@ddobrigk ddobrigk enabled auto-merge (squash) December 14, 2024 20:02
@github-actions github-actions bot changed the title Common: QA developments, cent work for more estimators [Common] QA developments, cent work for more estimators Dec 14, 2024
@alibuild alibuild mentioned this pull request Dec 14, 2024
Merged
alibuild
alibuild previously approved these changes Dec 14, 2024
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @ddobrigk.

@ddobrigk ddobrigk disabled auto-merge December 14, 2024 20:09
@ddobrigk ddobrigk enabled auto-merge (squash) December 14, 2024 20:09
alibuild
alibuild previously approved these changes Dec 14, 2024
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @ddobrigk.

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for 1350325 at 2024-12-16 00:59:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/8999-slc9_x86-64/0/Common/LegacyDataQA/otfv0qa.cxx:52:38: error: unused parameter 'col' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@ddobrigk ddobrigk disabled auto-merge December 21, 2024 19:21
@ddobrigk ddobrigk enabled auto-merge (squash) December 21, 2024 19:21
alibuild
alibuild approved these changes Dec 21, 2024
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @ddobrigk.

@ddobrigk ddobrigk merged commit e0ad248 into AliceO2Group:master Dec 21, 2024
10 of 12 checks passed
feisenhu pushed a commit to feisenhu/O2Physics that referenced this pull request Jan 8, 2025
@ddobrigk @alibuild @feisenhu
[Common] QA developments, cent work for more estimators (AliceO2Group…
c796246 
…#8999)

Co-authored-by: ALICE Builder <[email protected]>
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Jan 11, 2025
@ddobrigk @alibuild
[Common] QA developments, cent work for more estimators (AliceO2Group…
a2fedc0 
…#8999)

Co-authored-by: ALICE Builder <[email protected]>
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Feb 17, 2025
@ddobrigk @alibuild @smaff92
[Common] QA developments, cent work for more estimators (AliceO2Group…
42a3cc4 
…#8999)

Co-authored-by: ALICE Builder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alibuild alibuild alibuild approved these changes

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

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

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

Assignees
No one assigned
Labels
common datamodel
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ddobrigk @alibuild