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

[PWGHF] Add MC Generated ThnSparse in Lc task #8887

Merged
merged 2 commits into from
Dec 9, 2024
Merged

[PWGHF] Add MC Generated ThnSparse in Lc task #8887

merged 2 commits into from
Dec 9, 2024

Conversation

zhangbiao-phy
Copy link
Collaborator

  • Add MC Generated ThnSparse into the task
  • Complete the ptB and tracklets into the task

@github-actions github-actions bot added the pwghf PWG-HF label Dec 9, 2024
@zhangbiao-phy zhangbiao-phy marked this pull request as ready for review December 9, 2024 16:54
mfaggin
mfaggin reviewed Dec 9, 2024
View reviewed changes
PWGHF/D2H/Tasks/taskLc.cxx Outdated
@@ -514,6 +534,14 @@ struct HfTaskLc {
continue;
}
auto ptGen = particle.pt();
auto originType = particle.originMcGen();
auto ptGenB = -1;
unsigned numPvContributors = 0;
Copy link
Collaborator

Choose a reason for hiding this comment

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

do you mean unsigned int?

Copy link
Collaborator Author

@zhangbiao-phy zhangbiao-phy Dec 9, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

yes, it's shorthand for unsigned int, but let me complete it for more readability, Thanks!

@mfaggin mfaggin enabled auto-merge (squash) December 9, 2024 17:08
@mfaggin mfaggin merged commit 6358059 into AliceO2Group:master Dec 9, 2024
11 of 13 checks passed
Archita-Dash pushed a commit to Archita-Dash/O2Physics that referenced this pull request Dec 11, 2024
@zhangbiao-phy @Archita-Dash
[PWGHF] Add MC Generated ThnSparse in Lc task (AliceO2Group#8887)
ca10511
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mfaggin mfaggin mfaggin approved these changes

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

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

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

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

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

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

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

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

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

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

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

Assignees
No one assigned
Labels
pwghf PWG-HF
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zhangbiao-phy @mfaggin