Skip to content

[PWGLF,PWGMM] mftReassociationValidation.cxx : add features, rename some plots#15640

Merged
gbencedi merged 9 commits intoAliceO2Group:masterfrom
AlexianL:changes_mftReasso
Apr 4, 2026
Merged

[PWGLF,PWGMM] mftReassociationValidation.cxx : add features, rename some plots#15640
gbencedi merged 9 commits intoAliceO2Group:masterfrom
AlexianL:changes_mftReasso

Conversation

@AlexianL
Copy link
Copy Markdown
Contributor

@AlexianL AlexianL commented Apr 2, 2026 •
edited
Loading

In this PR, I added :

  • A cut on true tracks (with the McMask)
  • A cut to remove split vertices
  • An explicit cut on orphan tracks (even though they should be removed by definition)
  • Added a plot comparing ambDegree and compatibleCollIds.size
  • Added a precise track selection plot
  • Added the option to use only primary particle, only secondary particle and both
  • Added MFT track DCAxy and DCAz cut (with the z-vertex shift correction)
  • Added MFT track algorithm cut, allowing to choose only CA, only LTF or both
  • Plots of the DCA distribution w.r.t to the status of the track (MC matched or not)
  • A correction on some histogram names

@github-actions github-actions bot changed the title mftReassociationValidation.cxx : add features, rename some plots [PWGLF,PWGMM] mftReassociationValidation.cxx : add features, rename some plots Apr 2, 2026
@alibuild alibuild mentioned this pull request Apr 2, 2026
Closed
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026 •
edited
Loading

O2 linter results: ❌ 0 errors, ⚠️ 0 warnings, 🔕 0 disabled

@alibuild alibuild mentioned this pull request Apr 2, 2026
Closed
@AlexianL AlexianL force-pushed the changes_mftReasso branch from 2500567 to 94a633e Compare April 2, 2026 20:52
@AlexianL AlexianL marked this pull request as ready for review April 2, 2026 21:08
@AlexianL AlexianL force-pushed the changes_mftReasso branch from 9c33ca5 to 07a7fc9 Compare April 3, 2026 07:44
@AlexianL AlexianL force-pushed the changes_mftReasso branch from b8f9075 to 2f50acf Compare April 3, 2026 08:27
@gbencedi gbencedi merged commit 37738ef into AliceO2Group:master Apr 4, 2026
12 of 13 checks passed
vkucera
vkucera reviewed Apr 7, 2026
View reviewed changes
PWGMM/Mult/Tasks/mftReassociationValidation.cxx
<< " from its GRPMagField CCDB object";
o2::base::Propagator::initFieldFromGRP(grpmag);

o2::field::MagneticField* field = static_cast<o2::field::MagneticField*>(TGeoGlobalMagField::Instance()->GetField());
Copy link
Copy Markdown
Collaborator

@vkucera vkucera Apr 7, 2026

Choose a reason for hiding this comment

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

Include what you use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vkucera vkucera vkucera left review comments

@gbencedi gbencedi gbencedi approved these changes

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

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

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

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

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

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

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

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

Assignees

No one assigned

Labels

pwglf pwgmm

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AlexianL @vkucera @gbencedi