Skip to content

PWGHF: Speedup 3 prong combinatorics - #39

Merged
ginnocen merged 2 commits into
AliceO2Group:masterfrom
ktf:speedup-hf-3-prong-combinatorics
Aug 25, 2021
Merged

PWGHF: Speedup 3 prong combinatorics#39
ginnocen merged 2 commits into
AliceO2Group:masterfrom
ktf:speedup-hf-3-prong-combinatorics

Conversation

@ktf

@ktfktf commented Aug 25, 2021

Copy link
Copy Markdown
Member

No description provided.

ktf added 2 commits August 25, 2021 15:17
@ktf

ktf commented Aug 25, 2021

Copy link
Copy Markdown
MemberAuthor

@ginnocen at the moment what really seems to happen in PbPb is that we have either an infinite loop (for tiny tracks tables) or we are simply slow. This should protect the infinite loop and speed up things a bit. The next performance issue in this bit of code is the lookup by string in the Label2D.

@vkuceravkucera changed the title Speedup hf 3 prong combinatoricsPWGHF: Speedup 3 prong combinatoricsAug 25, 2021
@fgrosa

Copy link
Copy Markdown
Collaborator

hi @ktf thanks a lot! You could implement the same improvement for the speedup in the method for 2-prong preselections

voidis2ProngPreselected(constT1& hfTracks, T2& cutStatus, T3& whichHypo, int& isSelected)

or if you prefer I can do it.

@ktf

ktf commented Aug 25, 2021

Copy link
Copy Markdown
MemberAuthor

@fgrosa sure, I will, but in a separate PR.
@ginnocen@vkucera can you approve?

@ginnocen
ginnocen enabled auto-merge (squash) August 25, 2021 21:12
@ginnocen
ginnocen merged commit a94fb5a into AliceO2Group:masterAug 25, 2021
@ktf
ktf deleted the speedup-hf-3-prong-combinatorics branch August 25, 2021 21:55
@ktf

ktf commented Aug 26, 2021

Copy link
Copy Markdown
MemberAuthor

@fgrosa actually the two prong one is not so straight forward, so I will leave it to you.

alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Nov 28, 2023
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Apr 24, 2024
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Sep 5, 2024
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Oct 3, 2025
[PWGCF] Please consider the following formatting changes to AliceO2Group#13220
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 7, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 30, 2025
Sign up for freeto 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

@ktf@fgrosa@ginnocen