PWGHF: Ds-h correlations, minor adjustments in assoc particle efficiency evaluation

[scattaru]

Dear all,

• I've fixed the evaluation of associated particle efficiency requiring to use only particles coming from sel8 events for the reconstructed part (numerator).
• I've fixed the secondary particle contamination evaluation requiring that the Ds meson used to build the correlations is a real signal candidate.

Let me know if there are any comments,
Best regards,
Samuele

[scattaru]

Dear all,
Following what it was reported by @fgrosa in the HF O2 Analysis channel on the usage of sel8 in the processes which use MC samples not anchored I think that inserting the selection as I did the process will not work anymore. So I can add a configurable to choose if I want to apply the sel8 or not which seems to me the fast way at the moment or I can also remove that part. For the moment I would convert this PR in a draft.

Best regards,
Samuele

[scattaru]

Dear all,
the PR is ready for the review now. I have added a configurable (i.e. useSel8) in order to have the possibility to choose to select or not the collision requiring the sel8 selection. And I have done some little modifications in order to adapt the code for the evaluation of the systematic uncertainties of the analysis. Let me know for any comment.

Best regards,
Samuele

[fgrosa]

Hi @scattaru thanks a lot!
In principle it would be better to templetize the function in order to avoid the consumption of the EvSel table if the event selection is not applied (see for instance

O2Physics/PWGHF/TableProducer/trackIndexSkimCreator.cxx

Lines 197 to 208 in cc34780

	/// Event selection with trigger selection
	void processTrigSel(soa::Join<aod::Collisions, aod::EvSels>::iterator const& collision)
	{
	selectCollision<true, CentralityEstimator::None>(collision);
	}
	PROCESS_SWITCH(HfTrackIndexSkimCreatorTagSelCollisions, processTrigSel, "Use trigger selection", false);
	/// Event selection without trigger selection
	void processNoTrigSel(aod::Collision const& collision)
	{
	selectCollision<false, CentralityEstimator::None>(collision);
	}

). However, since this is needed for the SQM preliminaries I approve it (but consider the comment above for the future, thanks!).

[scattaru]

Hi @fgrosa,
Thanks a lot for the suggestion, in future I'll rewrite that part of the code following the way it is done in the TrackIndexSkimCreator

Thanks,
Samuele