Skip to content

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

Open
jjungIKF wants to merge 1 commit intoAliceO2Group:masterfrom
jjungIKF:master
Open

[PWGEM] LMee: Adding more DCA correlation histograms to various signals.#15602
jjungIKF wants to merge 1 commit 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.

@vkucera vkucera reopened this Apr 8, 2026
@vkucera vkucera marked this pull request as ready for review April 8, 2026 18:59
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