Skip to content

PWGHF: add sel8, PV posZ and TF border cut selections in candidateCreators.#5057

Merged
fgrosa merged 7 commits into
AliceO2Group:masterfrom
mfaggin:evselHfCollisions
Mar 10, 2024
Merged

PWGHF: add sel8, PV posZ and TF border cut selections in candidateCreators.#5057
fgrosa merged 7 commits into
AliceO2Group:masterfrom
mfaggin:evselHfCollisions

Conversation

@mfaggin

@mfaggin mfaggin commented Mar 8, 2024

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread PWGHF/Utils/utilsEvSelHf.h Outdated
Comment thread PWGHF/Utils/utilsEvSelHf.h Outdated
Comment thread PWGHF/Utils/utilsEvSelHf.h Outdated
@mfaggin

mfaggin commented Mar 8, 2024

Copy link
Copy Markdown
Collaborator Author

thanks a lot @vkucera , I've implemented you comments in the last commit

@vkucera

vkucera commented Mar 8, 2024

Copy link
Copy Markdown
Collaborator

thanks a lot @vkucera , I've implemented you comments in the last commit

Thanks @mfaggin , please also fix the isHfCollisionSelected documentation.

@vkucera vkucera left a comment

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.

Looks good to me!

@fgrosa fgrosa merged commit d67f942 into AliceO2Group:master Mar 10, 2024
@mfaggin mfaggin deleted the evselHfCollisions branch March 11, 2024 07:44
hahassan7 pushed a commit to hahassan7/O2Physics that referenced this pull request May 17, 2024
…ators. (AliceO2Group#5057)

* Define ev. sel. function, and add it to candidateCreator3Prong.

* Add ev. sel. to candidateCreator3Prong.

* Add ev. sel. to candidateCreatorDstar.

* Add ev. sel. to candidateCreatorCascade.

* Fix MegaLinter.

* Implement comments by Vit.

* Adjust isHfCollisionSelected documentation.

---------

Co-authored-by: Mattia Faggin <mfaggin@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants