From f43c1e75479ffd6b81610c42c70a2024fb22e29f Mon Sep 17 00:00:00 2001 From: SCHOTTER Romain <47983209+romainschotter@users.noreply.github.com> Date: Fri, 11 Sep 2026 17:14:36 +0200 Subject: [PATCH 1/3] Add configurable for old covariance matrix calculation --- PWGLF/Utils/strangenessBuilderModule.h | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/PWGLF/Utils/strangenessBuilderModule.h b/PWGLF/Utils/strangenessBuilderModule.h index 59e94a379d8..e77f784a2b4 100644 --- a/PWGLF/Utils/strangenessBuilderModule.h +++ b/PWGLF/Utils/strangenessBuilderModule.h @@ -334,6 +334,9 @@ struct coreConfigurables : o2::framework::ConfigurableGroup { // test the possibility of refitting with material corrections (DCA Fitter option) o2::framework::Configurable refitWithMaterialCorrection{"refitWithMaterialCorrection", false, "do refit after material corrections were applied"}; + + // enable old convariance matrix calculation until this has been fully tested + o2::framework::Configurable useOldCovMatrixCalculation{"useOldCovMatrixCalculation", true, "Use old covariance matrix calculation?"}; }; // strangenessBuilder: V0 building options @@ -824,6 +827,9 @@ class BuilderModule // Set option to refit with material corrections straHelper.fitter.setRefitWithMatCorr(baseOpts.refitWithMaterialCorrection.value); + // set option to enabled the old mode + straHelper.fitter.setOldMode(baseOpts.useOldCovMatrixCalculation.value); + // Initialise the RCTFlagsChecker if (eventSelectOpts.cfgApplyRCTrequirement) { rctFlagsChecker.init(eventSelectOpts.cfgRCTLabel.value, eventSelectOpts.cfgCheckZDC, eventSelectOpts.cfgTreatLimitedAcceptanceAsBad); From 0a9bac74db683aa396bd0fc7ca9b1660fd65da92 Mon Sep 17 00:00:00 2001 From: SCHOTTER Romain <47983209+romainschotter@users.noreply.github.com> Date: Fri, 11 Sep 2026 17:15:08 +0200 Subject: [PATCH 2/3] Set default value of old mode in the DCAfitter --- PWGLF/Utils/strangenessBuilderHelper.h | 1 + 1 file changed, 1 insertion(+) diff --git a/PWGLF/Utils/strangenessBuilderHelper.h b/PWGLF/Utils/strangenessBuilderHelper.h index c78f6471de9..6d5b525dcd4 100644 --- a/PWGLF/Utils/strangenessBuilderHelper.h +++ b/PWGLF/Utils/strangenessBuilderHelper.h @@ -253,6 +253,7 @@ class strangenessBuilderHelper fitter.setMaxChi2(1e9); fitter.setUseAbsDCA(true); fitter.setWeightedFinalPCA(false); + fitter.setOldMode(true); // enable old convariance matrix calculation until this has been fully tested v0selections.minCrossedRows = -1; v0selections.dcanegtopv = -1.0f; From fcafa6e5c578433ca6e4d6a84c4c3ffe6f51ae0a Mon Sep 17 00:00:00 2001 From: SCHOTTER Romain <47983209+romainschotter@users.noreply.github.com> Date: Fri, 11 Sep 2026 18:21:33 +0200 Subject: [PATCH 3/3] Rename configurable to avoid confusion --- PWGLF/Utils/strangenessBuilderModule.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PWGLF/Utils/strangenessBuilderModule.h b/PWGLF/Utils/strangenessBuilderModule.h index e77f784a2b4..648b8b45661 100644 --- a/PWGLF/Utils/strangenessBuilderModule.h +++ b/PWGLF/Utils/strangenessBuilderModule.h @@ -336,7 +336,7 @@ struct coreConfigurables : o2::framework::ConfigurableGroup { o2::framework::Configurable refitWithMaterialCorrection{"refitWithMaterialCorrection", false, "do refit after material corrections were applied"}; // enable old convariance matrix calculation until this has been fully tested - o2::framework::Configurable useOldCovMatrixCalculation{"useOldCovMatrixCalculation", true, "Use old covariance matrix calculation?"}; + o2::framework::Configurable useOldModeDCAFitter{"useOldModeDCAFitter", true, "Use old mode for DCA fitter?"}; }; // strangenessBuilder: V0 building options @@ -828,7 +828,7 @@ class BuilderModule straHelper.fitter.setRefitWithMatCorr(baseOpts.refitWithMaterialCorrection.value); // set option to enabled the old mode - straHelper.fitter.setOldMode(baseOpts.useOldCovMatrixCalculation.value); + straHelper.fitter.setOldMode(baseOpts.useOldModeDCAFitter.value); // Initialise the RCTFlagsChecker if (eventSelectOpts.cfgApplyRCTrequirement) {