Skip to content

PWGHF: further optimise HF combinatorial#4539

Merged
mfaggin merged 6 commits intoAliceO2Group:masterfrom
fgrosa:hfskim
Feb 1, 2024
Merged

PWGHF: further optimise HF combinatorial#4539
mfaggin merged 6 commits intoAliceO2Group:masterfrom
fgrosa:hfskim

Conversation

@fgrosa
Copy link
Collaborator

@fgrosa fgrosa commented Jan 31, 2024

This PR adds two optimisations for the HF combinatorial:

  • For Lc->pK0S, the bachelor track is filtered considering the proton PID (no other hypotheses are used)
  • For 3-prong channels, it adds the possibility to apply a cut on the decay length computed with the first two tracks to avoid the third loop

fgrosa added a commit to fgrosa/O2Physics that referenced this pull request Jan 31, 2024
@fgrosa
Merge pull request #57 from alibuild/alibot-cleanup-4539
f06fd34 
Please consider the following formatting changes to AliceO2Group#4539
@fgrosa fgrosa marked this pull request as draft January 31, 2024 21:01
fgrosa added a commit to fgrosa/O2Physics that referenced this pull request Jan 31, 2024
@fgrosa
Merge pull request #58 from alibuild/alibot-cleanup-4539
cfa243e 
Please consider the following formatting changes to AliceO2Group#4539
@fgrosa fgrosa marked this pull request as ready for review January 31, 2024 21:42
mfaggin
mfaggin approved these changes Feb 1, 2024
View reviewed changes
Copy link
Collaborator

@mfaggin mfaggin left a comment

Choose a reason for hiding this comment

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

thanks @fgrosa I approve it

@mfaggin mfaggin merged commit 5964514 into AliceO2Group:master Feb 1, 2024
basiach pushed a commit to basiach/O2Physics that referenced this pull request Feb 23, 2024
@fgrosa
PWGHF: further optimise HF combinatorial (AliceO2Group#4539)
b6035d6 
* PWGHF: further optimise HF combinatorial

* Handle the case in which the D0 decay vtx is not computed

* Fix cast in filter

* Add missing condition in if statements

* Add also chi2 PCA cut for 2-track vertices for 3-prongs

* Two minor fixes
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

@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

@nzardosh nzardosh Awaiting requested review from nzardosh

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

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fgrosa @mfaggin