Skip to content

[PWGDQ] Update dqEfficiency_withAssoc.cxx with 2 missing lines for efficienc…#13224

Closed
shreyasiacharya wants to merge 3 commits into
AliceO2Group:masterfrom
shreyasiacharya:master
Closed

[PWGDQ] Update dqEfficiency_withAssoc.cxx with 2 missing lines for efficienc…#13224
shreyasiacharya wants to merge 3 commits into
AliceO2Group:masterfrom
shreyasiacharya:master

Conversation

@shreyasiacharya

Copy link
Copy Markdown
Contributor

…y table

@github-actions github-actions Bot added the pwgdq label Oct 3, 2025
@github-actions github-actions Bot changed the title Update dqEfficiency_withAssoc.cxx with 2 missing lines for efficienc… [PWGDQ] Update dqEfficiency_withAssoc.cxx with 2 missing lines for efficienc… Oct 3, 2025
@github-actions

github-actions Bot commented Oct 3, 2025

Copy link
Copy Markdown

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

@vkucera vkucera marked this pull request as draft October 3, 2025 16:18
@vkucera

vkucera commented Oct 3, 2025

Copy link
Copy Markdown
Collaborator

@shreyasiacharya Please do not open PRs as "ready for review" which do not compile and fail other required checks.

@shreyasiacharya shreyasiacharya marked this pull request as ready for review October 3, 2025 16:20
Please consider the following formatting changes to #13224
@vkucera vkucera marked this pull request as draft October 3, 2025 16:27
@vkucera

vkucera commented Oct 3, 2025

Copy link
Copy Markdown
Collaborator

Why did you mark it as "ready for review" again? Your PR does not compile.

@alibuild

alibuild commented Oct 4, 2025

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 60de211 at 2025-10-04 11:45:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/13224-slc9_x86-64/0/PWGDQ/Tasks/dqEfficiency_withAssoc.cxx:1298:3: error: declaration does not declare anything [-fpermissive]
/sw/SOURCES/O2Physics/13224-slc9_x86-64/0/PWGDQ/Tasks/dqEfficiency_withAssoc.cxx:2251:13: error: 'MCTruthTableEffi' was not declared in this scope
ninja: build stopped: subcommand failed.

Full log here.

Produces<aod::DileptonsInfo> dileptonInfoList;
Produces<aod::JPsieeCandidates> PromptNonPromptSepTable;
Produces<aod::OniaMCTruth> MCTruthTableEffi;
Produces<aod::OniaMCTruth>;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why did you remove the name of the table to be produced?

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.

4 participants