The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
| Tag Class | Total number of occurrences | Tag strings used by tag class |
|---|---|---|
| Open issues in the code | 222 | TODO, FIXME, XXX |
Each tag is detailed below:
Number of occurrences found in the code: 222
| org.opentrafficsim.graphs.ContourPlotTest | Line |
|---|---|
| @Test | 96 |
| @Test | 115 |
| @Test | 174 |
| @Test | 193 |
| looks wrong / PK | 523 |
| org.opentrafficsim.graphs.FundamentalDiagram | Line |
| --No comment-- | 619 |
| org.opentrafficsim.graphs.TrajectoryPlot | Line |
| ensure that shapes for all the data points don't get allocated. Currently JFreeChart allocates many megabytes of memory for Ellipses that are never drawn. | 325 |
| lane change causes error... | 576 |
| org.opentrafficsim.graphs.TrajectoryPlotTest | Line |
| adapt to new path (List<Lane>) concept | 38 |
| we take the first (and only) lane on which the vehicle is registered. | 177 |
| org.opentrafficsim.road.car.CarTest | Line |
| check with following model as part of tactical planner assertEquals("The gtu following model should be " + gtuFollowingModel, gtuFollowingModel, referenceCar .getBehavioralCharacteristics().getGTUFollowingModel()); There is (currently) no way to retrieve the lane change model of a GTU. | 104 |
| org.opentrafficsim.road.gtu.AbstractLaneBasedGTUTest | Line |
| Test with gfm as part of tactical planner assertEquals("GTU following model should be identical to the provided one", gfm, car .getBehavioralCharacteristics().getGTUFollowingModel()); | 145 |
| removeLane should throw an Error when the car is not on that lane (currently this is silently ignored) | 393 |
| figure out why the added lane has a non-zero position | 394 |
| org.opentrafficsim.road.gtu.LaneBasedGTUTest | Line |
| see how we can ask the vehicle to look this far ahead | 181 |
| see how we can ask the vehicle to look this far behind | 189 |
| see how we can ask the vehicle to look 'forwardMaxDistance' ahead | 223 |
| follower = truck.headway(reverseMaxDistance); | 246 |
| org.opentrafficsim.road.gtu.LaneBasedTemplateGTUTypeTest | Line |
| assertTrue("Passengers cars are allowed on a no trucks lane", passengerCar.isCompatible(trucksForbidden)); | 300 |
| assertFalse("Trucks are not allowed on a no trucks lane", truck.isCompatible(trucksForbidden)); | 301 |
| assertFalse("Passenger cars are not allowed on a trucks only lane", passengerCar.isCompatible(trucksOnly)); | 302 |
| assertTrue("Trucks are allowed on a trucks only lane", truck.isCompatible(trucksOnly)); | 303 |
| assertTrue("Passenger cars are allowed on an urban road", passengerCar.isCompatible(urbanRoad)); | 304 |
| assertTrue("Trucks are allowed on an urban road", truck.isCompatible(urbanRoad)); | 305 |
| assertFalse("Passenger cars are not allowed on a bicycle path", passengerCar.isCompatible(bicycleLane)); | 306 |
| assertFalse("Trucks are not allowed on an urban road", truck.isCompatible(bicycleLane)); | 307 |
| org.opentrafficsim.road.gtu.animation.DefaultCarAnimation | Line |
| this could be a property of a GTU | 134 |
| org.opentrafficsim.road.gtu.animation.DefaultSwitchableGTUColorer | Line |
| LaneChangeUrgeGTUColorer COLORERS[3] = new LaneChangeUrgeGTUColorer(new Length(10, LengthUnit.METER), new Length(1000, LengthUnit.METER)); | 38 |
| org.opentrafficsim.road.gtu.generator.AbstractGTUGenerator | Line |
| different from this.direction? | 247 |
| THIS METHOD IS ALSO IN PERCEPTION -- DON'T DUPLICATE; ALSO, THIS VERSION IS WRONG. | 309 |
| Only follow links on the Route if there is a "real" Route if (routeNavigator.getRoute() == null || routeNavigator.getRoute().size() == 0 /* XXXXX STUB dummy route */ || routeNavigator.getRoute().containsLink((Link) theLane.getParentLink())) | 333 |
| XX STUB dummy route | 334 |
| THIS METHOD IS ALSO IN PERCEPTION -- DON'T DUPLICATE; ALSO, THIS VERSION IS RONG. | 370 |
| Only follow links on the Route if there is a "real" Route if (routeNavigator.getRoute() == null || routeNavigator.getRoute().size() == 0 /* XXXXX STUB dummy route */ || routeNavigator.getRoute().containsLink((Link) theLane.getParentLink())) | 394 |
| XX STUB dummy route | 395 |
| org.opentrafficsim.road.gtu.generator.ListGTUGenerator | Line |
| use given position in the constructor? | 169 |
| use given directionality in the constructor? | 172 |
| org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU | Line |
| LaneBasedGTU should KNOW its reference lane! | 163 |
| LaneBasedGTU should KNOW its reference lane! | 304 |
| WRONG - should be based on timeAtPosition() as the plan can have acc/dec/const segments | 511 |
| extra argument for DIR_MINUS driving direction? | 525 |
| look if more lanes are entered in one timestep, and continue the algorithm with the remainder of the time... | 532 |
| extra argument for DIR_MINUS driving direction? | 564 |
| extra argument for DIR_MINUS driving direction? | 601 |
| look if more lanes are exited in one timestep, and continue the algorithm with the remainder of the time... | 623 |
| XXXXXXXXXXXXXXX Minus one ULP is not safe if you want to add the current time | 638 |
| XXXXXXXXXXXXXXX Should compute the time time at which the rear of the GTU exits the lane??? getSimulator().scheduleEventRel(new Duration(timestep - Math.ulp(timestep), TimeUnit.SI), this, this, "leaveLane", new Object[] { lane, new Boolean(true) }); // TODO should be false? | 639 |
| should be false? | 641 |
| should be false? | 656 |
| direction dependent... <br> | 663 |
| Hack - this should be done more considerate -- fails at loops... | 699 |
| direction dependent... <br> | 723 |
| Hack - this should be done more considerate -- fails at loops... | 759 |
| LaneBasedGTU should KNOW its reference lane! | 855 |
| org.opentrafficsim.road.gtu.lane.LaneBasedIndividualGTU | Line |
| Should throw a more specific Exception type | 494 |
| org.opentrafficsim.road.gtu.lane.changing.LaneChangeModelTest | Line |
| create a LinkAnimation if the simulator is compatible with that. | 94 |
| The current LinkAnimation is too bad to use... | 95 |
| Decide what type of overtaking conditions we want in this test | 125 |
| test/prove the expected differences between Egoistic and Altruistic | 311 |
| prove that the most restrictive car in the other lane determines what happens | 312 |
| test merge into overtaking lane | 313 |
| org.opentrafficsim.road.gtu.lane.object.AbstractCSEObject | Line |
| REPAIR | 46 |
| org.opentrafficsim.road.gtu.lane.object.LaneBlock | Line |
| use animationClass | 58 |
| org.opentrafficsim.road.gtu.lane.object.LaneBlockOnOff | Line |
| --No comment-- | 60 |
| org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception | Line |
| update lane structure | 75 |
| create lane structure | 80 |
| possibly optimize by using a 'singleton' lane structure source | 83 |
| org.opentrafficsim.road.gtu.lane.perception.categories.DefaultAlexander | Line |
| In other places in OTS LEFT is positive (and RIGHT is negative). This should be made more consistent. | 614 |
| end of lanepath | 713 |
| allow observation of other objects as well. | 841 |
| --No comment-- | 940 |
| calculate real overlaps | 953 |
| expand for other types of Headways | 1014 |
| org.opentrafficsim.road.gtu.lane.perception.categories.InfrastructurePerception | Line |
| remove this if-statement and its contents | 100 |
| implement this method | 226 |
| implement this method | 252 |
| implement this method | 271 |
| remove this if-statement and its contents | 307 |
| org.opentrafficsim.road.gtu.lane.perception.categories.NeighborsPerception | Line |
| remove this line | 81 |
| use for-loop with working lane structure instead of this dummy code | 94 |
| remove this if-statement and its contents | 121 |
| remove this if-statement and its contents | 161 |
| remove this if-statement and its contents | 198 |
| remove this if-statement and its contents | 359 |
| break search, but how to guarantee that the rear of further GTU's is not within lookahead? | 426 |
| remove this if-statement and its contents | 446 |
| break search, but how to guarantee that the front of further GTU's is not within lookback? | 515 |
| enable the check below once the lane structure operates | 718 |
| remove code below | 735 |
| org.opentrafficsim.road.gtu.lane.perception.headway.HeadwayGTUReal | Line |
| obtain speed limit info from GTU | 64 |
| obtain speed limit info from GTU | 82 |
| org.opentrafficsim.road.gtu.lane.perception.headway.HeadwayGTUType | Line |
| create SpeedLimitInfo on the basis of this.gtuTypeAssumptions.getLaneTypeMaxSpeed(...) | 142 |
| org.opentrafficsim.road.gtu.lane.perceptionold.AbstractLanePerception | Line |
| private LaneStructure laneStructure; | 99 |
| end of lanepath | 375 |
| allow observation of other objects as well. | 503 |
| --No comment-- | 599 |
| calculate real overlaps | 612 |
| In other places in OTS LEFT is positive (and RIGHT is negative). This should be made more consistent. | 668 |
| expand for other types of Headways | 720 |
| org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder | Line |
| driving with negative speeds (backward driving) is not yet supported. <p> Copyright (c) 2013-2016 Delft University of Technology, PO Box 5, 2600 AA, Delft, the Netherlands. All rights reserved. <br> BSD-style license. See <a href="http://opentrafficsim.org/docs/license.html">OpenTrafficSim License</a>. </p> $LastChangedDate: 2015-07-24 02:58:59 +0200 (Fri, 24 Jul 2015) $, @version $Revision: 1147 $, by $Author: averbraeck $, initial version Nov 15, 2015 <br> | 36 |
| also for other driving directions, additional arguments in projectFractional? | 353 |
| org.opentrafficsim.road.gtu.lane.tactical.AbstractLaneBasedTacticalPlanner | Line |
| widest lane in case we are registered on more than one lane with the reference point | 111 |
| lane that leads to our location or not if we are registered on parallel lanes? | 112 |
| lane closest to length or 0 | 117 |
| this method can probably disappear (lots of duplicated code) | 327 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedCFLCTacticalPlanner | Line |
| skip if: - we are in the right lane and drive at max speed or we accelerate maximally - there are no other lanes | 151 |
| whether we drive on the right should be stored in some central place. | 159 |
| Make a 100% lateral move from standing still... | 256 |
| make a gradual lateral move | 260 |
| move laneIncentives to LanePerception? Figure out if the default lane incentives are OK, or override them with values that should keep this GTU on the intended route. | 311 |
| these comparisons to -10 is ridiculous. | 367 |
| --No comment-- | 444 |
| move suitability to LanePerception? Return the suitability for the current lane, left adjacent lane or right adjacent lane. | 479 |
| --No comment-- | 521 |
| this method should probably be moved into the CrossSectionLink class | 823 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedGTUFollowingChange0TacticalPlanner | Line |
| make direction dependent! | 164 |
| we are driving... | 179 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 187 |
| we are driving... | 230 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 238 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 410 |
| make type of plan (Egoistic, Altruistic) parameter of the class | 422 |
| make the elasticities 2.0 and 0.1 parameters of the class | 424 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedGTUFollowingLaneChangeTacticalPlanner | Line |
| we are driving... | 186 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 194 |
| we are driving... | 233 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 240 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 397 |
| do not understamd why this is added... Length maxDistance = lanePathInfo.getPath().getLength().minus(gtu.getLength().multiplyBy(2.0)); sameLaneTraffic.add(new HeadwayGTU("ENDPATH", Speed.ZERO, maxDistance, GTUType.NONE)); otherLaneTraffic.add(new HeadwayGTU("ENDPATH", Speed.ZERO, maxDistance, GTUType.NONE)); | 408 |
| if we move from standstill, create a longer plan, e.g. 4-5 seconds, with high acceleration! | 413 |
| make type of plan (Egoistic, Altruistic) parameter of the class | 414 |
| make the elasticities 2.0 and 0.1 parameters of the class | 416 |
| take the widest (now a random one) | 429 |
| if the distance is too small, do not build a path. Minimum = 0.5 * vehicle length | 444 |
| this should be solved in the time domain, not in the distance domain... | 445 |
| arbitrary... | 446 |
| can forwardHeadway still return null? | 451 |
| arbitrary... | 478 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedGTUFollowingTacticalPlanner | Line |
| I really don't like this -- if there is a lane drop at 20 m, the GTU should stop... | 83 |
| org.opentrafficsim.road.gtu.lane.tactical.following.AbstractGTUFollowingModelMobil | Line |
| Under which circumstances can getDistance() be NULL? Should that indeed result in TOODANGEROUS? | 61 |
| org.opentrafficsim.road.gtu.lane.tactical.following.FixedAccelerationModel | Line |
| should be specified in constructor | 88 |
| org.opentrafficsim.road.gtu.lane.tactical.following.IDMOld | Line |
| maxDistance dV is the approach speed | 109 |
| org.opentrafficsim.road.gtu.lane.tactical.following.IDMPlusMulti | Line |
| add vehicle length corresponding to key 'headway' | 59 |
| org.opentrafficsim.road.gtu.lane.tactical.following.IDMPlusOld | Line |
| maxDistance | 116 |
| org.opentrafficsim.road.gtu.lane.tactical.following.SequentialFixedAccelerationModel | Line |
| incorporate stepSize | 155 |
| org.opentrafficsim.road.gtu.lane.tactical.lanechangemobil.AbstractLaneChangeModel | Line |
| make this driving side dependent; i.e. implement a general way to figure out on which side of the road cars are supposed to drive | 59 |
| org.opentrafficsim.road.gtu.lane.tactical.lanechangemobil.LaneChangeModel | Line |
| Lane change models should use the perceived nearby GTUs rather than a provided list of same lane traffic, etc. <p> Copyright (c) 2013-2016 Delft University of Technology, PO Box 5, 2600 AA, Delft, the Netherlands. All rights reserved. <br> BSD-style license. See <a href="http://opentrafficsim.org/docs/license.html">OpenTrafficSim License</a>. <p> | 15 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.IncentiveCourtesy | Line |
| XX STUB | 34 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.IncentiveHierarchal | Line |
| XX STUB | 31 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRS | Line |
| traffic lights on route, possible on different lane (and possibly close) | 139 |
| org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo | Line |
| vehicle not ahead and not backwards but completely adjacent | 113 |
| remove this test code | 165 |
| infinite -> some limited space, speed also one leader | 369 |
| definition of 'right most lane' does not account for ramps and weaving sections | 675 |
| org.opentrafficsim.road.gtu.lane.tactical.toledo.ToledoCarFollowing | Line |
| speed difference with reaction time | 140 |
| org.opentrafficsim.road.gtu.lane.tactical.toledo.ToledoFactory | Line |
| remove LOOKBACKOLD and insert LOOKBACK once NeighborsPerception uses LOOKBACK | 33 |
| org.opentrafficsim.road.gtu.lane.tactical.toledo.ToledoPerception | Line |
| updateInfrastructureLaneChangeInfo with split number | 31 |
| org.opentrafficsim.road.gtu.lane.tactical.util.ConflictUtil | Line |
| plus? confused about rear overlap for two other things... tailway? {@formatter:off} ______________________________________________ ___ virtual headway | ___ | |___|(-----------------------)|___|(vehicle from south, on lane from south) _____________________________|_______|________ / / / / {@formatter:on} | 213 |
| respond to indicator / expected turn, will the conflict vehicle go over the conflict? Consider further conflicting vehicles if not. | 339 |
| disregard conflicting vehicles with a route not over the conflict, if known through e.d. indicator | 389 |
| all-stop behavior | 533 |
| org.opentrafficsim.road.gtu.lane.tactical.util.lmrs.LmrsUtil | Line |
| updated headway based on (estimated) desire of other vehicle uses current headway, this results in larger deceleration than using the desire we require a desire where T = desire*Tmin + (1-desire)*Tmax gives a deceleration of b*desire | 172 |
| headway of other driver... | 239 |
| headway of other driver... | 253 |
| org.opentrafficsim.road.network.LinkLocationTest | Line |
| distance to location on another link (not yet possible; currently ALWAYS returns null) | 70 |
| org.opentrafficsim.road.network.factory.XMLNetworkGeneratorTest | Line |
| repair headway in such a way that vehicle does not have to brake (safe distance) | 129 |
| org.opentrafficsim.road.network.factory.xml.FillTag | Line |
| GTUColorer | 198 |
| create a FILL | 225 |
| GTUMix | 227 |
| RouteMix | 228 |
| ShortestRoute | 229 |
| ShortestRouteMix | 230 |
| org.opentrafficsim.road.network.factory.xml.GTUColorerTag | Line |
| use parameters for AccelerationGTUColorer | 79 |
| default colorer | 93 |
| org.opentrafficsim.road.network.factory.xml.GeneratorTag | Line |
| parse direction | 148 |
| GTUMix | 307 |
| RouteMix | 308 |
| ShortestRoute | 309 |
| ShortestRouteMix | 310 |
| Different strategical planner factories | 311 |
| org.opentrafficsim.road.network.factory.xml.GlobalTag | Line |
| add other GTUColorer tags | 54 |
| parse other GTUColorer tags | 108 |
| org.opentrafficsim.road.network.factory.xml.Links | Line |
| test for over-specification (i.e. node direction was already specified) | 232 |
| move the radius if there is an start and end offset? How? | 427 |
| Map<GTUType, LongitudinalDirectionality> linkDirections = new HashMap<>(); | 509 |
| correct? | 558 |
| correct? | 575 |
| Quick hack to solve the error that the lane directionality has not (yet) been registered at the link | 652 |
| LISTGENERATOR | 731 |
| org.opentrafficsim.road.network.factory.xml.ListGeneratorTag | Line |
| GTUColorer | 134 |
| org.opentrafficsim.road.network.factory.xml.NodeTag | Line |
| slope as an angle. | 48 |
| slope for the Node. | 84 |
| org.opentrafficsim.road.network.factory.xml.XmlNetworkLaneParser | Line |
| shortestRoute, routeMix, ShortestRouteMix | 189 |
| Make routes GTU specific. See what to do with GTUType.ALL for routes | 212 |
| org.opentrafficsim.road.network.lane.CrossSectionElement | Line |
| use throwIf | 151 |
| org.opentrafficsim.road.network.lane.CurveTest | Line |
| finish writing this test | 112 |
| org.opentrafficsim.road.network.lane.Lane | Line |
| allow for direction-dependent speed limit | 91 |
| allow for direction-dependent sensors | 98 |
| allow for direction-dependent overtaking conditions | 131 |
| constructor calls with this(...) | 320 |
| take the cross section slices into account... | 399 |
| take the cross section slices into account... | 413 |
| take the cross section slices into account... | 436 |
| take the cross section slices into account... | 450 |
| make sure triggering is done right when driving in DIR_MINUS direction | 644 |
| only center position? Or also width? What is a good cutoff? Base on average width of the GTU type that can drive on this Lane? E.g., for a Tram or Train, a 5 cm deviation is a problem; for a Car or a Bicycle, more deviation is acceptable. | 830 |
| And is it aligned with its next lane? | 876 |
| And is it aligned with its next lane? | 890 |
| And is it aligned with its next lane? | 948 |
| And is it aligned with its next lane? | 962 |
| org.opentrafficsim.road.network.lane.LaneTest | Line |
| what overtaking conditions do we ant to test in this unit test? | 83 |
| what overtaking conditions do we ant to test in this unit test? | 140 |
| what overtaking conditions do we ant to test in this unit test? | 157 |
| what overtaking conditions do we ant to test in this unit test? | 220 |
| org.opentrafficsim.road.network.lane.SensorTest | Line |
| this is not true anymore with OperationalPlans, Perception, etc => | 147 |
| the number of events that should be scheduled can vary per models chosen | 148 |
| assertEquals("There should be three scheduled events (trigger, leaveLane, | 149 |
| car.move, terminate)", 4, eventList.size()); The sensor should be triggered around t=38.3403 (exact value: 10 / 9 * (sqrt(3541) - 25)) System.out.println("trigger event is " + triggerEvent); / TODO not triggered in next half second. | 150 |
| assertEquals("Trigger event should be around 38.3403", 38.3403, | 154 |
| triggerEvent.getAbsoluteExecutionTime().get().getSI(), 0.0001); | 155 |
| check that the sensor is triggered at the right time. | 182 |
| org.opentrafficsim.road.network.speed.package-info | Line |
| --No comment-- | 19 |