From b8f152cfc1911ab40b2984462159dcfcef06be78 Mon Sep 17 00:00:00 2001 From: Yannick Brosseau Date: Fri, 3 Nov 2023 09:27:47 -0400 Subject: [PATCH] Don't copie the scenario filter into the commonparameters When creating the initial ConnectionSet, we fetch the filters directly from the scenario object. With the current code copying the filter data, we would reapply the same filters to the trips list in the reset functions, spending time for no use. --- .../src/parameters/common_parameters.cpp | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/connection_scan_algorithm/src/parameters/common_parameters.cpp b/connection_scan_algorithm/src/parameters/common_parameters.cpp index d59a693..7723ea8 100644 --- a/connection_scan_algorithm/src/parameters/common_parameters.cpp +++ b/connection_scan_algorithm/src/parameters/common_parameters.cpp @@ -27,16 +27,7 @@ namespace TrRouting maxFirstWaitingTimeSeconds(maxFirstWaitingTime), forwardCalculation(forward) { - scenarioUuid = scenario.uuid; - onlyServices = scenario.servicesList; - onlyLines = scenario.onlyLines; - onlyAgencies = scenario.onlyAgencies; - onlyNodes = scenario.onlyNodes; - onlyModes = scenario.onlyModes; - exceptLines = scenario.exceptLines; - exceptAgencies = scenario.exceptAgencies; - exceptNodes = scenario.exceptNodes; - exceptModes = scenario.exceptModes; + scenarioUuid = scenario.uuid; //TODO Check if this is used somewhere } CommonParameters::CommonParameters(const CommonParameters& baseParams):