Skip to content

[PWGEM] LMee: Adding more DCA correlation histograms to various signals.#15602

Draft
jjungIKF wants to merge 2 commits intoAliceO2Group:masterfrom
jjungIKF:master
Draft

[PWGEM] LMee: Adding more DCA correlation histograms to various signals.#15602
jjungIKF wants to merge 2 commits intoAliceO2Group:masterfrom
jjungIKF:master

Conversation

@jjungIKF
Copy link
Copy Markdown
Contributor

No description provided.

@dsekihat
Copy link
Copy Markdown
Collaborator

dsekihat commented Apr 2, 2026

Hi Jerome, It looks good to me. But, please resolve conflicts.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026

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

@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented Apr 2, 2026

Error while checking build/O2Physics/o2 for aa27f41 at 2026-04-02 14:30:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/Dilepton/Core/DileptonMC.h:528:103: error: 'kTH2F' was not declared in this scope; did you mean 'o2::framework::kTH2F'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/Dilepton/Core/DileptonMC.h:549:110: error: 'kTH2F' was not declared in this scope; did you mean 'o2::framework::kTH2F'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/Dilepton/Core/DileptonMC.h:584:110: error: 'kTH2F' was not declared in this scope; did you mean 'o2::framework::kTH2F'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/Dilepton/Core/DileptonMC.h:528:103: error: 'kTH2F' was not declared in this scope; did you mean 'o2::framework::kTH2F'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/Dilepton/Core/DileptonMC.h:549:110: error: 'kTH2F' was not declared in this scope; did you mean 'o2::framework::kTH2F'?
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGEM/Dilepton/Core/DileptonMC.h:584:110: error: 'kTH2F' was not declared in this scope; did you mean 'o2::framework::kTH2F'?
ninja: build stopped: subcommand failed.

Full log here.

@vkucera vkucera marked this pull request as draft April 2, 2026 13:00
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Apr 2, 2026

@jjungIKF You are supposed to test your branch before pushing.

@jjungIKF
Copy link
Copy Markdown
Contributor Author

jjungIKF commented Apr 2, 2026

@vkucera my original commit was before your commit changing all the name spaces was merged

@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Apr 2, 2026

@vkucera my original commit was before your commit changing all the name spaces was merged

And then you updated your branch without testing it.

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