Skip to content

Conversation

@jikim1290
Copy link
Contributor

adding ddbar histograms for normalization
separating useraxis for the same and mixing containers

@victor-gonzalez
Copy link
Collaborator

@jaelpark Green light from your side?

@jaelpark
Copy link
Collaborator

jaelpark commented Dec 9, 2024

Looks okay to me. But I think if the new merge rules require the code authors, then it's fair to add all the other names too. Particularly @jgrosseo, who's the original author. I mostly just added the 2-prong stuff and some other additions.

@jikim1290
Copy link
Contributor Author

@victor-gonzalez Thanks for checking the confirmation, @jaelpark Thanks for the comment. I did not get that. I have added @jgrosseo . Many thanks for @jgrosseo for this very nice framework

@victor-gonzalez victor-gonzalez merged commit 10255a0 into AliceO2Group:master Dec 9, 2024
10 checks passed
Archita-Dash pushed a commit to Archita-Dash/O2Physics that referenced this pull request Dec 11, 2024
…8789)

Co-authored-by: junleekim <junleekim@maccbookjunlee2024.dyndns.cern.ch>
Co-authored-by: junleekim <junleekim@Mac.lan>
@vkucera
Copy link
Collaborator

vkucera commented Dec 11, 2024

@jikim1290 Why did you silence the name/workflow-file check?

hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Dec 20, 2024
…8789)

Co-authored-by: junleekim <junleekim@maccbookjunlee2024.dyndns.cern.ch>
Co-authored-by: junleekim <junleekim@Mac.lan>
feisenhu pushed a commit to feisenhu/O2Physics that referenced this pull request Jan 8, 2025
…8789)

Co-authored-by: junleekim <junleekim@maccbookjunlee2024.dyndns.cern.ch>
Co-authored-by: junleekim <junleekim@Mac.lan>
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Feb 17, 2025
…8789)

Co-authored-by: junleekim <junleekim@maccbookjunlee2024.dyndns.cern.ch>
Co-authored-by: junleekim <junleekim@Mac.lan>
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