From 9d45f37cef8a8eff2ab8e48e744ff386677bfb8c Mon Sep 17 00:00:00 2001 From: Kai Nagel Date: Mon, 22 Apr 2024 08:25:34 +0200 Subject: [PATCH] mostly comments; some refactoring --- .../RunOneTaxiWithPrebookingExampleIT.java | 16 +-- .../mobsim/qsim/PreplanningEngineTest.java | 77 ++++++++++++ .../drtAndPt/PtAlongALine2Test.java | 9 +- .../mobsim/qsim/ActivityEngineWithWakeup.java | 5 +- .../core/mobsim/qsim/PreplanningEngine.java | 116 +++++++++++------- .../core/mobsim/qsim/PreplanningUtils.java | 15 +++ .../components/QSimComponentsConfigGroup.java | 9 ++ 7 files changed, 191 insertions(+), 56 deletions(-) create mode 100644 contribs/vsp/src/test/java/org/matsim/core/mobsim/qsim/PreplanningEngineTest.java create mode 100644 matsim/src/main/java/org/matsim/core/mobsim/qsim/PreplanningUtils.java diff --git a/contribs/dvrp/src/test/java/org/matsim/contrib/dvrp/examples/onetaxi/RunOneTaxiWithPrebookingExampleIT.java b/contribs/dvrp/src/test/java/org/matsim/contrib/dvrp/examples/onetaxi/RunOneTaxiWithPrebookingExampleIT.java index b6661027c45..a5d8baff1cf 100644 --- a/contribs/dvrp/src/test/java/org/matsim/contrib/dvrp/examples/onetaxi/RunOneTaxiWithPrebookingExampleIT.java +++ b/contribs/dvrp/src/test/java/org/matsim/contrib/dvrp/examples/onetaxi/RunOneTaxiWithPrebookingExampleIT.java @@ -47,16 +47,14 @@ import org.matsim.contrib.dvrp.run.DvrpModule; import org.matsim.contrib.dvrp.run.DvrpQSimComponents; import org.matsim.api.core.v01.events.HasPersonId; +import org.matsim.contrib.otfvis.OTFVisLiveModule; import org.matsim.core.config.Config; import org.matsim.core.config.ConfigGroup; import org.matsim.core.config.ConfigUtils; import org.matsim.core.controler.AbstractModule; import org.matsim.core.controler.Controler; import org.matsim.core.events.handler.BasicEventHandler; -import org.matsim.core.mobsim.qsim.AbstractQSimModule; -import org.matsim.core.mobsim.qsim.ActivityEngineModule; -import org.matsim.core.mobsim.qsim.ActivityEngineWithWakeup; -import org.matsim.core.mobsim.qsim.PreplanningEngine; +import org.matsim.core.mobsim.qsim.*; import org.matsim.core.mobsim.qsim.components.QSimComponentsConfigGroup; import org.matsim.core.scenario.ScenarioUtils; import org.matsim.core.utils.io.IOUtils; @@ -96,9 +94,7 @@ void testRun() { Scenario scenario = ScenarioUtils.loadScenario(config); for (Person person : scenario.getPopulation().getPersons().values()) { - person.getSelectedPlan() - .getAttributes() - .putAttribute(PreplanningEngine.PREBOOKING_OFFSET_ATTRIBUTE_NAME, 900.); + PreplanningUtils.setPrebookingOffset_s( person.getSelectedPlan(), 900. ); } //PopulationUtils.writePopulation(scenario.getPopulation(), utils.getOutputDirectory() + "/../pop.xml"); @@ -158,6 +154,12 @@ public void install() { } }); + if ("true".equals(System.getProperty("runOTFVis"))) { + // This will start otfvis + controler.addOverridingModule(new OTFVisLiveModule() ); + // !! does not work together with parameterized tests :-( !! + } + // run simulation controler.run(); diff --git a/contribs/vsp/src/test/java/org/matsim/core/mobsim/qsim/PreplanningEngineTest.java b/contribs/vsp/src/test/java/org/matsim/core/mobsim/qsim/PreplanningEngineTest.java new file mode 100644 index 00000000000..74536deae58 --- /dev/null +++ b/contribs/vsp/src/test/java/org/matsim/core/mobsim/qsim/PreplanningEngineTest.java @@ -0,0 +1,77 @@ +package org.matsim.core.mobsim.qsim; + +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.RegisterExtension; +import org.matsim.api.core.v01.Scenario; +import org.matsim.api.core.v01.population.Person; +import org.matsim.contrib.otfvis.OTFVisLiveModule; +import org.matsim.core.config.Config; +import org.matsim.core.config.ConfigUtils; +import org.matsim.core.controler.Controler; +import org.matsim.core.mobsim.qsim.components.QSimComponentsConfigGroup; +import org.matsim.core.scenario.ScenarioUtils; +import org.matsim.core.utils.io.IOUtils; +import org.matsim.examples.ExamplesUtils; +import org.matsim.testcases.MatsimTestUtils; + +import javax.inject.Singleton; + +class PreplanningEngineTest{ + // Cannot use otfvis if in core matsim. --> In vsp contrib for time being. kai, apr'24 + private static final Logger log = LogManager.getLogger(PreplanningEngineTest.class ); + + @RegisterExtension private MatsimTestUtils utils = new MatsimTestUtils(); + + @Test public void test() { + // I am interested here in testing this NOT with DRT but with other modes. kai, apr'24 + // In the somewhat longer run, should work together with fleetpy (of TUM). kai, apr'24 + + Config config = ConfigUtils.loadConfig( IOUtils.extendUrl( ExamplesUtils.getTestScenarioURL( "equil" ), "config.xml" ) ); + + config.controller().setOutputDirectory( utils.getOutputDirectory() ); + config.controller().setLastIteration( 0 ); + + QSimComponentsConfigGroup componentsConfig = ConfigUtils.addOrGetModule( config, QSimComponentsConfigGroup.class ); + + componentsConfig.removeActiveComponent( ActivityEngineModule.COMPONENT_NAME ); + componentsConfig.addActiveComponent( ActivityEngineWithWakeup.COMPONENT_NAME ); + + componentsConfig.addActiveComponent( PreplanningEngineQSimModule.COMPONENT_NAME ); + + Scenario scenario = ScenarioUtils.loadScenario( config ); + for( Person person : scenario.getPopulation().getPersons().values() ){ + PreplanningUtils.setPrebookingOffset_s( person.getSelectedPlan(), 900. ); + } + + Controler controler = new Controler( scenario ); + + controler.addOverridingQSimModule( new AbstractQSimModule(){ + @Override protected void configureQSim(){ + bind(PreplanningEngine.class).asEagerSingleton(); + addQSimComponentBinding(PreplanningEngineQSimModule.COMPONENT_NAME).to(PreplanningEngine.class).in( Singleton.class ); + // Does the following: + // * on prepare sim: register all departure handlers that implement the TripInfo.Provider interface + // * + + // the above just installs the functionality; it also needs to be requested (from config). + + // needs to go along with ActivityEngineWithWakeup. complains if not bound. + // yy is, however, ok if bound but not activated. --?? --> should end up in same module! + + addQSimComponentBinding( ActivityEngineWithWakeup.COMPONENT_NAME ).to( ActivityEngineWithWakeup.class ).in( Singleton.class ); + + } + } ); + + if ("true".equals(System.getProperty("runOTFVis"))) { + // This will start otfvis if property is set + controler.addOverridingModule(new OTFVisLiveModule() ); + // !! does not work together with parameterized tests :-( !! + } + + controler.run(); + } + +} diff --git a/contribs/vsp/src/test/java/org/matsim/integration/drtAndPt/PtAlongALine2Test.java b/contribs/vsp/src/test/java/org/matsim/integration/drtAndPt/PtAlongALine2Test.java index 7d884443580..3a1e217d4f4 100644 --- a/contribs/vsp/src/test/java/org/matsim/integration/drtAndPt/PtAlongALine2Test.java +++ b/contribs/vsp/src/test/java/org/matsim/integration/drtAndPt/PtAlongALine2Test.java @@ -45,10 +45,7 @@ import org.matsim.core.controler.Controler; import org.matsim.core.controler.events.IterationEndsEvent; import org.matsim.core.controler.listener.IterationEndsListener; -import org.matsim.core.mobsim.qsim.AbstractQSimModule; -import org.matsim.core.mobsim.qsim.ActivityEngineModule; -import org.matsim.core.mobsim.qsim.ActivityEngineWithWakeup; -import org.matsim.core.mobsim.qsim.PreplanningEngine; +import org.matsim.core.mobsim.qsim.*; import org.matsim.core.mobsim.qsim.components.QSimComponentsConfigGroup; import org.matsim.core.router.AnalysisMainModeIdentifier; import org.matsim.core.router.TripStructureUtils; @@ -321,9 +318,7 @@ void testPtAlongALineWithRaptorAndDrtServiceArea() { } if (drtMode == DrtMode.withPrebooking) { for (Person person : scenario.getPopulation().getPersons().values()) { - person.getSelectedPlan() - .getAttributes() - .putAttribute(PreplanningEngine.PREBOOKING_OFFSET_ATTRIBUTE_NAME, 7200.); + PreplanningUtils.setPrebookingOffset_s( person.getSelectedPlan(), 7200. ); } } diff --git a/matsim/src/main/java/org/matsim/core/mobsim/qsim/ActivityEngineWithWakeup.java b/matsim/src/main/java/org/matsim/core/mobsim/qsim/ActivityEngineWithWakeup.java index b23b695b9d4..64d9e9a3319 100644 --- a/matsim/src/main/java/org/matsim/core/mobsim/qsim/ActivityEngineWithWakeup.java +++ b/matsim/src/main/java/org/matsim/core/mobsim/qsim/ActivityEngineWithWakeup.java @@ -26,6 +26,8 @@ import jakarta.inject.Inject; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.matsim.api.core.v01.Id; import org.matsim.api.core.v01.events.Event; import org.matsim.api.core.v01.population.Activity; @@ -37,7 +39,7 @@ public final class ActivityEngineWithWakeup implements ActivityEngine { public static final String COMPONENT_NAME = "ActivityEngineWithWakeup"; - + private static final Logger log = LogManager.getLogger(ActivityEngineWithWakeup.class ); private final EventsManager eventsManager; private final PreplanningEngine preplanningEngine; private final ActivityEngine delegate; @@ -55,6 +57,7 @@ public final class ActivityEngineWithWakeup implements ActivityEngine { @Override public void onPrepareSim() { + log.warn( "running onPrepareSim"); delegate.onPrepareSim(); } diff --git a/matsim/src/main/java/org/matsim/core/mobsim/qsim/PreplanningEngine.java b/matsim/src/main/java/org/matsim/core/mobsim/qsim/PreplanningEngine.java index 0f34e2d4a5b..5232b826ccf 100644 --- a/matsim/src/main/java/org/matsim/core/mobsim/qsim/PreplanningEngine.java +++ b/matsim/src/main/java/org/matsim/core/mobsim/qsim/PreplanningEngine.java @@ -72,11 +72,12 @@ import com.google.inject.Inject; public final class PreplanningEngine implements MobsimEngine { - // Could implement this as a generalized version of the bdi-abm implementation: can send notifications to agent, and agent can react. Similar to the drive-to action. - // Notifications and corresponding handlers could then be registered. On the other hand, it is easy to add an engine such as this one; how much does it help to have another - // layer of infrastructure? Am currently leaning towards the second argument. kai, mar'19 + // Could implement this as a generalized version of the bdi-abm implementation: can send notifications to agent, and agent can react. Similar + // to the drive-to action. Notifications and corresponding handlers could then be registered. - public static final String PREBOOKING_OFFSET_ATTRIBUTE_NAME = "prebookingOffset_s"; + // On the other hand, it is easy to add an engine such as this one; how much does it help to have another layer of infrastructure? + + // Am currently leaning towards the second argument. kai, mar'19 private static final Logger log = LogManager.getLogger(PreplanningEngine.class); @@ -93,9 +94,11 @@ public final class PreplanningEngine implements MobsimEngine { private final Map> tripInfoUpdatesMap = new TreeMap<>(comparing(Identifiable::getId )); // yyyy not sure about possible race conditions here! kai, feb'19 // yyyyyy can't have non-sorted maps here because we will get non-deterministic results. kai, mar'19 + // (haven't these two points be fixed by using the "comparing"? kai, apr'24) private final Map tripInfoRequestMap = new TreeMap<>(comparing(Identifiable::getId )); // yyyyyy can't have non-sorted maps here because we will get non-deterministic results. kai, mar'19 + // (hasn't this point be fixed by using the "comparing"? kai, apr'24) private EditPlans editPlans; @@ -113,6 +116,7 @@ public final class PreplanningEngine implements MobsimEngine { } @Override public void onPrepareSim() { + log.warn( "running onPrepareSim"); for (DepartureHandler departureHandler : internalInterface.getDepartureHandlers()) { if (departureHandler instanceof TripInfo.Provider) { String mode = ((TripInfo.Provider)departureHandler).getMode(); @@ -136,7 +140,8 @@ public final class PreplanningEngine implements MobsimEngine { // (I have inlined the below methods since I find this for the time being easier to read. Can be extracted again at some later point in time . // kai, jan'20) - // process all (initial) requests: + // the following goes through all requests (generated by the agent wakups), send them to the trip info providers, and decide based on + // the returned information: for (Map.Entry entry : tripInfoRequestMap.entrySet()) { final MobsimAgent mobsimAgent = entry.getKey(); final TripInfo.Request request = entry.getValue(); @@ -149,11 +154,14 @@ public final class PreplanningEngine implements MobsimEngine { // TODO add info for mode that is in agent plan, if not returned by trip info provider // not sure if that is needed. kai, jan'20 + // the following method decides, and + // * puts it then into the tripInfoUpdatesMap (processed below); or + // * if the agent needs to wait for confirmation, the confirming method (currently only in PassengerEngineWithPrebooking) puts it into tripInfoUpdatesMap. decide( mobsimAgent, allTripInfos ); } tripInfoRequestMap.clear(); - // process all updates: + // process the tripInfoUpdatesMap (see above): for (Map.Entry> entry : tripInfoUpdatesMap.entrySet()) { MobsimAgent agent = entry.getKey(); @@ -163,6 +171,14 @@ public final class PreplanningEngine implements MobsimEngine { TripInfo actualTripInfo = tripInfo.get(); updateAgentPlan(agent, actualTripInfo); } else { + // this can e.g. happen if the booking failed. At first glance, I am not too happy about this. I think that if a + // provider is not able to process the trip, it should not return a TripInfo offer. At second glance, it may happen + // that no provider returns a TripInfo offer. Now evidently, this could be avoided by always allowing for a fallback + // mode, e.g. walk and/or pt. On the other hand, since the functionality is already here, we can as well try to keep + // it. However, need to make sure that it eventually gets resolved. Also see the questions below. + + // in principle, one could always give the pt TripOption. However, this would be fairly expensive to compute. + TripInfo.Request request = null; //TODO get it from where ??? from TripInfo??? //TODO agent should adapt trip info request given that the previous one got rejected?? @@ -175,8 +191,10 @@ public final class PreplanningEngine implements MobsimEngine { // (I have inlined the above methods since I find this for the time being easier to read. Can be extracted again at some later point in time . kai, jan'20) } - public synchronized final void notifyChangedTripInformation(MobsimAgent agent, Optional tripInfoUpdate) { + public synchronized void notifyChangedTripInformation( MobsimAgent agent, Optional tripInfoUpdate ) { // yyyy My IDE complains about "Optional" in method signatures. kai, jan'20 + // It looks like it needs to be possible to return an "empty" tripInfoUpdate in order to notify that the "decided" (= selected) trip + // option did not work out. Or, alternatively, none of the providers returned an answer at all. tripInfoUpdatesMap.put(agent, tripInfoUpdate); } @@ -195,7 +213,7 @@ private void decide(MobsimAgent agent, List allTripInfos) { } // to get started, we assume that we are only getting one drt option back. - // TODO: make complete + // yyyy TODO: evidently, this needs to be changed to mode choice between available modes. kai, apr'24 TripInfo tripInfo = allTripInfos.iterator().next(); if (tripInfo instanceof TripInfoWithRequiredBooking) { @@ -220,9 +238,15 @@ private void decide(MobsimAgent agent, List allTripInfos) { editPlans.rescheduleActivityEnd(agent); + // (if the trip requires booking, the booking confirmation comes later, so we need to delay the call to + // notifyChangedTripInformation until we have confirmation. The "notifyChangedTripInfo" is actually called from within dvrp (PassengerEngineWithPrebooking). ) } else { - notifyChangedTripInformation(agent, Optional.of(tripInfo));//no booking here + // if we do not have to wait for the booking confirmation, we can immediately compute and insert the trip. This is (once + // more) done by first collecting it into a container, and process the container later. To achieve thread safety, inserting + // it into the container is a "synchronized" method: + notifyChangedTripInformation(agent, Optional.of(tripInfo)); } + log.warn("---"); } @@ -232,12 +256,10 @@ List generateWakeups( MobsimAgent agent, do return Collections.emptyList(); } - - Double prebookingOffset_s = (Double)((PlanAgent)agent).getCurrentPlan().getAttributes().getAttribute(PREBOOKING_OFFSET_ATTRIBUTE_NAME); - // yyyy prebooking info needs to be in plan since it will not survive in the leg. :-( kai, jan'20 + final Double prebookingOffset_s = PreplanningUtils.getPrebookingOffset_s( ((PlanAgent) agent).getCurrentPlan() ); if (prebookingOffset_s == null) { - log.warn("The " + PREBOOKING_OFFSET_ATTRIBUTE_NAME + " is not set in the agent. No wakeup for prebooking will be generated."); + log.warn("The " + "prebookingOffset_s" + " is not set in the agent. No wakeup for prebooking will be generated." ); return Collections.emptyList(); } @@ -246,12 +268,16 @@ List generateWakeups( MobsimAgent agent, do for (String mode : new String[] { TransportMode.drt, TransportMode.taxi } ) { // (only do the following for drt and taxi yyyy which means it may fail for, say, "drt2". kai, apr'23) + // (not doing this for, say, pt, is fine, though. we could still have pt as fallback mode for drt/taxi.) + for (Leg drtLeg : EditPlans.findLegsWithModeInFuture(agent, mode )) { // (find the corresponding legs) final double prebookingTime = drtLeg.getDepartureTime().seconds() - prebookingOffset_s; if (prebookingTime < agent.getActivityEndTime()) { - // yyyy and here one sees that having this in the activity engine is not very practical + // (yyyy and here one sees that having this in the activity engine is not very practical) + + // ### the following inserts the preplanLeg (--> preplanTrip??), to be executed at wakeup: ### log.info("generating wakeup entry"); wakeups.add(new ActivityEngineWithWakeup.AgentEntry(agent, prebookingTime, (agent1, then) -> preplanLeg(agent1, then, drtLeg )) ); } @@ -299,6 +325,12 @@ private void preplanLeg( MobsimAgent agent, double now, Leg leg ) { //first simulate ActivityEngineWithWakeup and then PreplanningEngine --> decision process //in the same time step this.notifyTripInfoNeeded(agent, request); + + // (this enters the request into tripInfoRequestMap ... would be easier to read this if it was inlined ... but the method needs to be + // threadsafe and this is easier to achieve with a separate method. kai, apr'24) + + // (the tripInfoRequestMap will be processed in every time step. Not sure if we can enforce that this happens after processing the + // wakeups, so it may happen in the following time step (--???). This would contradict the comment above the method call. kai, apr'24 ) } private void updateAgentPlan(MobsimAgent agent, TripInfo tripInfo) { @@ -327,33 +359,47 @@ private void updateAgentPlan(MobsimAgent agent, TripInfo tripInfo) { } Gbl.assertNotNull(inputTrip); - List result = new ArrayList<>(); - inputTrip.getOriginActivity().setEndTime(tripInfo.getExpectedBoardingTime() - 900); // yyyy means for time being we always depart 15min before pickup. kai, mar'19 WithinDayAgentUtils.resetCaches(agent); log.warn("agentId=" + agent.getId() + " | newActEndTime=" + inputTrip.getOriginActivity() - .getEndTime() - .seconds()); + .getEndTime() + .seconds()); + + final List result = createDrtTripInclAccessEgress( tripInfo, inputTrip ); + + TripRouter.insertTrip(plan, inputTrip.getOriginActivity(), result, inputTrip.getDestinationActivity()); + + editPlans.rescheduleActivityEnd(agent); + // I don't think that this can ever do damage. + + log.warn("new plan for agentId=" + agent.getId()); + for (PlanElement planElement : plan.getPlanElements()) { + log.warn(planElement.toString()); + } + log.warn("---"); + + } + private List createDrtTripInclAccessEgress( TripInfo tripInfo, TripStructureUtils.Trip inputTrip ){ + // code below currently has taxi hardcoded but this is not necessary IMO. kai, apr'24 - // result.add( inputTrip.getOriginActivity() ) ; - // --- + List result = new ArrayList<>(); PopulationFactory pf = population.getFactory(); { - Facility fromFacility = FacilitiesUtils.toFacility(inputTrip.getOriginActivity(), facilities); + Facility fromFacility = FacilitiesUtils.toFacility( inputTrip.getOriginActivity(), facilities ); Facility toFacility = tripInfo.getPickupLocation(); double departureTime = tripInfo.getExpectedBoardingTime() - 900.; // always depart 15min before pickup List planElements = tripRouter.calcRoute(TransportMode.walk, fromFacility, - toFacility, departureTime, null, inputTrip.getTripAttributes()); + toFacility, departureTime, null, inputTrip.getTripAttributes() ); // not sure if this works for walk, but it should ... result.addAll(planElements); } { Activity act = pf.createActivityFromLinkId(createStageActivityType(TransportMode.taxi), - tripInfo.getPickupLocation().getLinkId()); + tripInfo.getPickupLocation().getLinkId() ); act.setMaximumDuration(0.); result.add(act); } @@ -362,45 +408,33 @@ private void updateAgentPlan(MobsimAgent agent, TripInfo tripInfo) { result.add(leg); Route route = pf.getRouteFactories() .createRoute(GenericRouteImpl.class, tripInfo.getPickupLocation().getLinkId(), - tripInfo.getDropoffLocation().getLinkId()); + tripInfo.getDropoffLocation().getLinkId() ); leg.setRoute(route); } { Activity act = pf.createActivityFromLinkId(createStageActivityType(TransportMode.taxi), - tripInfo.getDropoffLocation().getLinkId()); + tripInfo.getDropoffLocation().getLinkId() ); act.setMaximumDuration(0.); result.add(act); } { Facility fromFacility = tripInfo.getDropoffLocation(); - Facility toFacility = FacilitiesUtils.toFacility(inputTrip.getDestinationActivity(), facilities); + Facility toFacility = FacilitiesUtils.toFacility( inputTrip.getDestinationActivity(), facilities ); double expectedTravelTime; try { expectedTravelTime = tripInfo.getExpectedTravelTime(); } catch (Exception ee) { - expectedTravelTime = 15. - * 60; // using 15min as quick fix since dvrp refuses to provide this. kai, mar'19 + expectedTravelTime = 15. * 60; // using 15min as quick fix since dvrp refuses to provide this. kai, mar'19 } double departureTime = tripInfo.getExpectedBoardingTime() + expectedTravelTime; List planElements = tripRouter.calcRoute(TransportMode.walk, fromFacility, - toFacility, departureTime, null, inputTrip.getOriginActivity().getAttributes()); + toFacility, departureTime, null, inputTrip.getOriginActivity().getAttributes() ); result.addAll(planElements); } // result.add( inputTrip.getDestinationActivity() ) ; - - TripRouter.insertTrip(plan, inputTrip.getOriginActivity(), result, inputTrip.getDestinationActivity()); - - editPlans.rescheduleActivityEnd(agent); - // I don't think that this can ever do damage. - - log.warn("new plan for agentId=" + agent.getId()); - for (PlanElement planElement : plan.getPlanElements()) { - log.warn(planElement.toString()); - } - log.warn("---"); - + return result; } static String toString(TripInfo info) { diff --git a/matsim/src/main/java/org/matsim/core/mobsim/qsim/PreplanningUtils.java b/matsim/src/main/java/org/matsim/core/mobsim/qsim/PreplanningUtils.java new file mode 100644 index 00000000000..bfc117ca153 --- /dev/null +++ b/matsim/src/main/java/org/matsim/core/mobsim/qsim/PreplanningUtils.java @@ -0,0 +1,15 @@ +package org.matsim.core.mobsim.qsim; + +import org.matsim.api.core.v01.population.Plan; + +public class PreplanningUtils{ + private PreplanningUtils() {} // do not instantiate + public static Double getPrebookingOffset_s( Plan plan ){ + // yyyy prebooking info needs to be in plan since it will not survive in the leg. :-( kai, jan'20 + return (Double) plan.getAttributes().getAttribute( "prebookingOffset_s" ); + } + public static void setPrebookingOffset_s( Plan plan, double offset ){ + plan.getAttributes().putAttribute( "prebookingOffset_s", offset ); + // yyyy prebooking info needs to be in plan since it will not survive in the leg. :-( kai, jan'20 + } +} diff --git a/matsim/src/main/java/org/matsim/core/mobsim/qsim/components/QSimComponentsConfigGroup.java b/matsim/src/main/java/org/matsim/core/mobsim/qsim/components/QSimComponentsConfigGroup.java index 8878ebe4118..c21058da8b3 100644 --- a/matsim/src/main/java/org/matsim/core/mobsim/qsim/components/QSimComponentsConfigGroup.java +++ b/matsim/src/main/java/org/matsim/core/mobsim/qsim/components/QSimComponentsConfigGroup.java @@ -80,6 +80,15 @@ public void addActiveComponent( String component ) { // (doing this the indirect way because of the Set vs List discussion above. kai, apr'23 } + public void removeActiveComponent( String component ) { + // I need this so often that I am finally adding it here. kai, apr'24 + + List components = getActiveComponents(); + components.remove( component ); + setActiveComponents( components ); + // (doing this the indirect way because of the Set vs List discussion above. kai, apr'24 + } + @StringGetter(ACTIVE_COMPONENTS) public String getActiveComponentsAsString() { return String.join(", ", activeComponents);