Skip to content

DPG/PWGHF: monitor single-track efficiency from HF decays#6260

Merged
mfaggin merged 5 commits into
AliceO2Group:masterfrom
mfaggin:qaEffHF
May 24, 2024
Merged

DPG/PWGHF: monitor single-track efficiency from HF decays#6260
mfaggin merged 5 commits into
AliceO2Group:masterfrom
mfaggin:qaEffHF

Conversation

@mfaggin
Copy link
Copy Markdown
Collaborator

@mfaggin mfaggin commented May 23, 2024

@fgrosa @fcolamar @gluparel @scattaru @mmazzilli @deepathoms this should help for the analysis of Samuele.
I still want to add the same code for the case w/o grouping by mcCollisions, I'll add it tomorrow and then I'll open the PR

@scattaru
Copy link
Copy Markdown
Contributor

Hi @mfaggin,
Thanks a lot for this update!

Cheers,
Samuele

@mfaggin
Copy link
Copy Markdown
Collaborator Author

mfaggin commented May 24, 2024

In this PR I add the possibility to calculate the efficiency for tracks from HF decays.
This implied some changes:

  • in processMC I needed to insert manually the loop over McCollisions grouping by hand the Collisions. This allows me to have the ungrouped McParticles table, avoiding unpredictable crashes in getCharmHadronOrigin (for b tracks, it might happen that the track is assigned to the wrong reco. collision, and if this collision belongs to a mcCollision different from the correct one for the beauty track and we work with grouped mcCollisions, then we miss the decay chain and the method fails)
  • I also added the possibility to cut on the Z coordinate of the generated vertex (up to now, only a cut on the reco. posZ was applied)
  • I replied the code in processMCWithoutCollisions. This duplication can be useful to study how the efficiency change if we ignore the track association to collisions (in processMC the efficiency is computed considering gen. particles only for reco. collisions, i.e beign a pure tracking efficiency indeed)

Tagging @fgrosa @fcolamar @scattaru @gluparel @deepathoms @mmazzilli in case they have further comments.
Tagging also @njacazio just to check whether I am not doing silly mistakes (btw from some check locally it seems ok)

@mfaggin mfaggin marked this pull request as ready for review May 24, 2024 09:45
@mfaggin mfaggin enabled auto-merge (squash) May 24, 2024 15:27
Copy link
Copy Markdown
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @mfaggin.

@mfaggin mfaggin merged commit 7b5749e into AliceO2Group:master May 24, 2024
@mfaggin mfaggin deleted the qaEffHF branch May 24, 2024 15:27
echizzal pushed a commit to echizzal/O2Physics that referenced this pull request Jul 18, 2024
…up#6260)

* Loop on McParticles and manual collision grouping.

* Add possibility to check only HF particles.

* Add possibility to apply cut on vertex-z for gen. collision.

* Add the code also in processMCWithoutCollisions.

* Fix clang format and MegaLinter.

---------

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