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 | 184 | TODO, FIXME, XXX |
Each tag is detailed below:
Number of occurrences found in the code: 184
| org.opentrafficsim.graphs.ContourPlotTest | Line |
|---|---|
| @Test | 98 |
| @Test | 118 |
| @Test | 178 |
| @Test | 198 |
| looks wrong / PK | 519 |
| org.opentrafficsim.graphs.FundamentalDiagram | Line |
| --No comment-- | 622 |
| org.opentrafficsim.graphs.FundamentalDiagramPlotTest | Line |
| @Test | 77 |
| THIS TEST FAILS!!! assertEquals("Density should be " + expectedDensity, expectedDensity, value, 0.00001); | 176 |
| 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. | 399 |
| lane change causes error... | 847 |
| org.opentrafficsim.graphs.TrajectoryPlotTest | Line |
| we take the first (and only) lane on which the vehicle is registered. | 221 |
| 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. | 105 |
| 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 | 136 |
| 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.LaneBasedGTUGenerator | Line |
| ALL? we need to stop for all gtus... | 188 |
| look beyond splitting lane | 199 |
| this is a hack, what if the reference position is not the middle? | 228 |
| 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 |
| widest lane in case we are registered on more than one lane with the reference point | 519 |
| lane that leads to our location or not if we are registered on parallel lanes? | 520 |
| extra argument for DIR_MINUS driving direction? | 555 |
| PK: this goes very wrong for networks with short lanes with alternating GTUDireationality | 570 |
| extra argument for DIR_MINUS driving direction? | 594 |
| this does not account for the GTUDirectionality next lane | 603 |
| do this at the right time, scheduled with triggers. 4x | 626 |
| extra argument for DIR_MINUS driving direction? | 638 |
| look if more lanes are exited in one timestep, and continue the algorithm with the remainder of the time... | 660 |
| XXXXXXXXXXXXXXX Minus one ULP is not safe if you want to add the current time | 675 |
| 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? | 676 |
| should be false? | 678 |
| should be false? | 693 |
| direction dependent... <br> | 700 |
| Hack - this should be done more considerate -- fails at loops... | 736 |
| direction dependent... <br> | 760 |
| Hack - this should be done more considerate -- fails at loops... | 796 |
| org.opentrafficsim.road.gtu.lane.LaneBasedIndividualGTU | Line |
| Should throw a more specific Exception type | 497 |
| org.opentrafficsim.road.gtu.lane.changing.LaneChangeModelTest | Line |
| create a LinkAnimation if the simulator is compatible with that. | 98 |
| The current LinkAnimation is too bad to use... | 99 |
| Decide what type of overtaking conditions we want in this test | 127 |
| test/prove the expected differences between Egoistic and Altruistic | 303 |
| prove that the most restrictive car in the other lane determines what happens | 304 |
| test merge into overtaking lane | 305 |
| org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception | Line |
| possibly optimize by using a 'singleton' lane structure source, per GTUType | 118 |
| possibly build and destroy at edges only | 119 |
| org.opentrafficsim.road.gtu.lane.perception.categories.DefaultSimplePerception | Line |
| In other places in OTS LEFT is positive (and RIGHT is negative). This should be made more consistent. | 647 |
| end of lanepath | 750 |
| allow observation of other objects as well. | 922 |
| --No comment-- | 1018 |
| calculate real overlaps | 1031 |
| expand for other types of Headways | 1092 |
| org.opentrafficsim.road.gtu.lane.perception.categories.InfrastructurePerception | Line |
| should be sink on routes, the above destination node method does not work if the route contains an extra node | 212 |
| implement this method, current implementation is a simple fix | 242 |
| splits | 278 |
| implement this method, lane map needs support for this (and legal lane changes) | 295 |
| org.opentrafficsim.road.gtu.lane.perception.categories.IntersectionPerception | Line |
| --No comment-- | 100 |
| org.opentrafficsim.road.gtu.lane.perception.categories.NeighborsPerception | Line |
| remove this line | 89 |
| remove this fix to ignore on-ramp block | 355 |
| break search, but how to guarantee that the rear of further GTU's is not within lookahead? | 367 |
| break search, but how to guarantee that the front of further GTU's is not within lookback? | 437 |
| more GTU statuses | 455 |
| not use this check when synchronizing or cooperating | 669 |
| remove code below | 687 |
| org.opentrafficsim.road.gtu.lane.perception.headway.AbstractHeadway | Line |
| fix this javadoc; there are obvious inconsistencies between this javadoc and the code below. | 107 |
| org.opentrafficsim.road.gtu.lane.perception.headway.HeadwayGTUReal | Line |
| this is a simple fix, and getReferenceLane() | 90 |
| org.opentrafficsim.road.gtu.lane.perception.headway.HeadwayGTUType | Line |
| create SpeedLimitInfo on the basis of this.gtuTypeAssumptions.getLaneTypeMaxSpeed(...) | 142 |
| 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> | 41 |
| also for other driving directions, additional arguments in projectFractional? | 355 |
| remove GTU argument (only needed for instantaneous lane change hack) | 543 |
| this elsewhere based on path | 567 |
| org.opentrafficsim.road.gtu.lane.tactical.AbstractLaneBasedTacticalPlanner | Line |
| this is wrong, the other direction may not be allowed, i.e. is not accessible | 621 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedCFLCTacticalPlanner | Line |
| how to handle objects on this lane or another lane??? | 131 |
| these two info's are not used | 146 |
| 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 |
| 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. | 240 |
| these comparisons to -10 is ridiculous. | 298 |
| move suitability to LanePerception? Return the suitability for the current lane, left adjacent lane or right adjacent lane. | 388 |
| --No comment-- | 611 |
| this method should probably be moved into the CrossSectionLink class | 668 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedGTUFollowingDirectedChangeTacticalPlanner | Line |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 183 |
| Object & GTU | 184 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 229 |
| GTU & Object | 230 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 405 |
| Object & GTU | 406 |
| make type of plan (Egoistic, Altruistic) parameter of the class | 417 |
| make the elasticities 2.0 and 0.1 parameters of the class | 419 |
| 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... | 84 |
| 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 | 111 |
| 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 | 117 |
| 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) | 146 |
| org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo | Line |
| vehicle not ahead and not backwards but completely adjacent | 112 |
| remove this test code | 164 |
| infinite -> some limited space, speed also one leader | 368 |
| definition of 'right most lane' does not account for ramps and weaving sections | 674 |
| 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} | 214 |
| respond to indicator / expected turn, will the conflict vehicle go over the conflict? Consider further conflicting vehicles if not. | 340 |
| disregard conflicting vehicles with a route not over the conflict, if known through e.d. indicator | 390 |
| all-stop behavior | 534 |
| org.opentrafficsim.road.gtu.lane.tactical.util.lmrs.LmrsUtil | Line |
| this is a hack to prevent right lane changes of all vehicles on the left lane when placed in network at t=0 | 163 |
| org.opentrafficsim.road.gtu.strategical.LaneBasedStrategicalRoutePlannerTest | Line |
| Gtu cannot be null anymore... LaneBasedStrategicalRoutePlanner lbsrp = new LaneBasedStrategicalRoutePlanner(bc, tacticalPlanner, null); | 60 |
| org.opentrafficsim.road.gtu.strategical.od.ODMatrix | Line |
| remove this method as soon as there is a JUNIT test | 489 |
| org.opentrafficsim.road.gtu.strategical.od.ODMatrixTrips | Line |
| pce (i.e. passenger car equivalent) instead of veh's | 27 |
| org.opentrafficsim.road.network.LinkLocationTest | Line |
| distance to location on another link (not yet possible; currently ALWAYS returns null) | 75 |
| org.opentrafficsim.road.network.lane.CrossSectionElement | Line |
| use throwIf | 135 |
| org.opentrafficsim.road.network.lane.CrossSectionLink | Line |
| work in a different way with lane numbers to align to standard lane numbering. | 52 |
| allow for the removal of a Lane; currently this is not possible. | 59 |
| org.opentrafficsim.road.network.lane.CurveTest | Line |
| finish writing this test | 111 |
| org.opentrafficsim.road.network.lane.Lane | Line |
| allow for direction-dependent speed limit | 103 |
| allow for direction-dependent sensors | 110 |
| allow for direction-dependent lane objects | 117 |
| allow for direction-dependent overtaking conditions | 150 |
| constructor calls with this(...) | 392 |
| take the cross section slices into account... | 471 |
| take the cross section slices into account... | 485 |
| take the cross section slices into account... | 508 |
| take the cross section slices into account... | 522 |
| make sure triggering is done right when driving in DIR_MINUS direction | 716 |
| 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. | 1062 |
| And is it aligned with its next lane? | 1108 |
| And is it aligned with its next lane? | 1122 |
| And is it aligned with its next lane? | 1180 |
| And is it aligned with its next lane? | 1194 |
| org.opentrafficsim.road.network.lane.LaneTest | Line |
| what overtaking conditions do we ant to test in this unit test? | 86 |
| what overtaking conditions do we ant to test in this unit test? | 139 |
| what overtaking conditions do we ant to test in this unit test? | 156 |
| 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 => | 149 |
| the number of events that should be scheduled can vary per models chosen | 150 |
| assertEquals("There should be three scheduled events (trigger, leaveLane, | 151 |
| 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. | 152 |
| assertEquals("Trigger event should be around 38.3403", 38.3403, | 156 |
| triggerEvent.getAbsoluteExecutionTime().get().getSI(), 0.0001); | 157 |
| check that the sensor is triggered at the right time. | 185 |
| org.opentrafficsim.road.network.lane.conflict.Conflict | Line |
| conflict needs to be connected to the other cloned conflict | 158 |
| org.opentrafficsim.road.network.lane.object.LaneBlock | Line |
| --No comment-- | 66 |
| org.opentrafficsim.road.network.lane.object.sensor.DirectionalOccupancySensor | Line |
| enforce clone method | 61 |
| org.opentrafficsim.road.network.lane.object.sensor.NonDirectionalOccupancySensor | Line |
| enforce clone method | 64 |
| org.opentrafficsim.road.network.lane.object.sensor.SensorAnimation | Line |
| this is strange... Why not Rectangle2D.Double(-this.halfWidth, -0.25, 2 * this.halfWidth, 0.5) ??? | 62 |
| org.opentrafficsim.road.network.lane.object.sensor.TrafficLightSensor | Line |
| Possibly provide the GTUTypes that trigger the sensor as an argument for the constructor | 87 |
| not handling backwards driving GTU | 255 |
| should the parent of the clone be our parent??? And should the (cloned) parent not construct its own flank sensors? | 504 |
| org.opentrafficsim.road.network.lane.object.sensor.TrafficLightSensorTest | Line |
| @Test | 190 |
| THIS TEST FAILS!! assertEquals("event list should contain one event (due to creation of the GTU on the detector)", 1, this.loggedEvents.size()); | 273 |
| org.opentrafficsim.road.network.lane.object.trafficlight.SimpleTrafficLight | Line |
| --No comment-- | 66 |
| org.opentrafficsim.road.network.sampling.RoadSampler | Line |
| GTUDirectionality from Lane.GTU_ADD_EVENT | 218 |
| Length from Lane.GTU_ADD_EVENT | 230 |
| GTUDirectionality from Lane.GTU_ADD_EVENT | 253 |
| Length from Lane.GTU_ADD_EVENT this doesn't work, as the GTU is no longer on the lane it was removed from try { position = gtu.position(lane, RelativePosition.REFERENCE_POSITION); } catch (GTUException exception) { throw new RuntimeException(exception); } | 258 |
| org.opentrafficsim.road.network.speed.package-info | Line |
| --No comment-- | 19 |