PWGHF: taskFlow.cxx: Add mixed-event process function for h-MFT #6177
Conversation
… multiplicity bins
AlexianL
requested review from
NicoleBastid,
alibuild,
deepathoms,
fcatalan92,
fcolamar,
fgrosa,
hahassan7,
jpxrk,
mfaggin,
mmazzilli,
nzardosh and
vkucera
as code owners
vkucera
changed the title
… changed the organization of the AnalysisResults root file
|
@NicoleBastid @mmazzilli @deepathoms could you please have a look? Thanks! |
mmazzilli
left a comment
mmazzilli
left a comment
There was a problem hiding this comment.
The PR is fine with me. My only suggestion is to use a better naming wrt "MyCollisions" and "MyTracks"
|
Hi Marianna, thanks a lot for checking the PR. I see that the change to "MyCollisions" was done in this HF-wide PR: #3454 . So maybe this is how it should be? I don't have a strong preference, so I can check how the naming should be and we can change it if needed. |
How about |
Hi Katarina, |
…eO2Group#6177) * Added h-MFT mixed event process function, trying to artificially fill multiplicity bins * taskFlow.cxx : adding mixed-event process function for h-MFT case * taskFlow.cxx : adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * Correct a mistake in the Mixed Event process function for h-MFT, also changed the organization of the AnalysisResults root file * Replaced MyCollisions and MyTracks by more comprehensive variable names * Attempt to fix megalinter issue by adding an empty line at the end of the file
…eO2Group#6177) * Added h-MFT mixed event process function, trying to artificially fill multiplicity bins * taskFlow.cxx : adding mixed-event process function for h-MFT case * taskFlow.cxx : adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * Correct a mistake in the Mixed Event process function for h-MFT, also changed the organization of the AnalysisResults root file * Replaced MyCollisions and MyTracks by more comprehensive variable names * Attempt to fix megalinter issue by adding an empty line at the end of the file
…eO2Group#6177) * Added h-MFT mixed event process function, trying to artificially fill multiplicity bins * taskFlow.cxx : adding mixed-event process function for h-MFT case * taskFlow.cxx : adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * PWGHF : taskFlow.cxx -> adding mixed-event process function for h-MFT case * Correct a mistake in the Mixed Event process function for h-MFT, also changed the organization of the AnalysisResults root file * Replaced MyCollisions and MyTracks by more comprehensive variable names * Attempt to fix megalinter issue by adding an empty line at the end of the file
5 participants
Few minor changes too : change of the default axisMass binning, typo correction, re-organization of the AnalysisResults root file, renaming of "MyCollisions" and "MyTracks" by more comprehensive names.