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 | 163 | TODO, FIXME, XXX |
Each tag is detailed below:
Number of occurrences found in the code: 163
| org.opentrafficsim.graphs.ContourPlotTest | Line |
|---|---|
| @Test | 98 |
| @Test | 118 |
| @Test | 178 |
| @Test | 198 |
| looks wrong / PK | 519 |
| org.opentrafficsim.graphs.FundamentalDiagram | Line |
| --No comment-- | 625 |
| 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 | 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 |
| shouldn't this loop over lanesToBeRemoved? | 259 |
| this is a simple fix, should be based on path | 274 |
| this is a simple fix, should be based on path | 297 |
| widest lane in case we are registered on more than one lane with the reference point | 485 |
| lane that leads to our location or not if we are registered on parallel lanes? | 486 |
| extra argument for DIR_MINUS driving direction? | 521 |
| look if more lanes are entered in one timestep, and continue the algorithm with the remainder of the time... | 528 |
| extra argument for DIR_MINUS driving direction? | 560 |
| do this at the right time, scheduled with triggers. 4x | 586 |
| extra argument for DIR_MINUS driving direction? | 598 |
| look if more lanes are exited in one timestep, and continue the algorithm with the remainder of the time... | 620 |
| XXXXXXXXXXXXXXX Minus one ULP is not safe if you want to add the current time | 635 |
| 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? | 636 |
| should be false? | 638 |
| should be false? | 653 |
| direction dependent... <br> | 660 |
| Hack - this should be done more considerate -- fails at loops... | 696 |
| direction dependent... <br> | 720 |
| Hack - this should be done more considerate -- fails at loops... | 756 |
| 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. | 97 |
| The current LinkAnimation is too bad to use... | 98 |
| Decide what type of overtaking conditions we want in this test | 128 |
| test/prove the expected differences between Egoistic and Altruistic | 315 |
| prove that the most restrictive car in the other lane determines what happens | 316 |
| test merge into overtaking lane | 317 |
| org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception | Line |
| possibly optimize by using a 'singleton' lane structure source, per GTUType | 116 |
| possibly build and destroy at edges only | 117 |
| org.opentrafficsim.road.gtu.lane.perception.LaneStructure | Line |
| --No comment-- | 324 |
| 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. | 608 |
| end of lanepath | 707 |
| allow observation of other objects as well. | 847 |
| --No comment-- | 943 |
| calculate real overlaps | 956 |
| expand for other types of Headways | 1017 |
| org.opentrafficsim.road.gtu.lane.perception.categories.InfrastructurePerception | Line |
| implement this method, current implementation is a simple fix | 216 |
| splits | 252 |
| implement this method, lane map needs support for this (and legal lane changes) | 269 |
| org.opentrafficsim.road.gtu.lane.perception.categories.IntersectionPerception | Line |
| SimpleTrafficLight is not yet a LaneBasedObject getPerception().getEnvironmentState().getSortedObjects(ViewingDirection.FORWARD, lane, SimpleTrafficLight.class); | 74 |
| needs a lot of input set.add(new HeadwayConflict(...)) | 104 |
| org.opentrafficsim.road.gtu.lane.perception.categories.NeighborsPerception | Line |
| remove this line | 82 |
| break search, but how to guarantee that the rear of further GTU's is not within lookahead? | 346 |
| break search, but how to guarantee that the front of further GTU's is not within lookback? | 410 |
| remove code below | 629 |
| 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() | 91 |
| 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? | 361 |
| remove GTU argument (only needed for instantaneous lane change hack) | 509 |
| this elsewhere based on path | 533 |
| org.opentrafficsim.road.gtu.lane.tactical.AbstractLaneBasedTacticalPlanner | Line |
| Specific tactical planners should know which perception categories to use SSMDEMO below categories can be disabled perception.addPerceptionCategory(new InfrastructurePerception(perception)); perception.addPerceptionCategory(new NeighborsPerception(perception)); perception.addPerceptionCategory(new IntersectionPerception(perception)); perception.addPerceptionCategory(new EgoPerception(perception)); | 79 |
| 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 | 145 |
| whether we drive on the right should be stored in some central place. | 153 |
| 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. | 207 |
| these comparisons to -10 is ridiculous. | 263 |
| move suitability to LanePerception? Return the suitability for the current lane, left adjacent lane or right adjacent lane. | 353 |
| this method should probably be moved into the CrossSectionLink class | 629 |
| 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} | 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 |
| 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... | 237 |
| headway of other driver... | 250 |
| 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 | 724 |
| 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. | 1070 |
| And is it aligned with its next lane? | 1116 |
| And is it aligned with its next lane? | 1130 |
| And is it aligned with its next lane? | 1188 |
| And is it aligned with its next lane? | 1202 |
| 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 => | 150 |
| the number of events that should be scheduled can vary per models chosen | 151 |
| assertEquals("There should be three scheduled events (trigger, leaveLane, | 152 |
| 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. | 153 |
| assertEquals("Trigger event should be around 38.3403", 38.3403, | 157 |
| triggerEvent.getAbsoluteExecutionTime().get().getSI(), 0.0001); | 158 |
| check that the sensor is triggered at the right time. | 186 |
| org.opentrafficsim.road.network.lane.object.LaneBlock | Line |
| --No comment-- | 63 |
| org.opentrafficsim.road.network.lane.object.sensor.TrafficLightSensor | Line |
| figure out how to detect GTUs that leave the sensor sideways | 59 |
| fire a sensor becomes occupied event | 131 |
| fire a sensor becomes unoccupied event | 140 |
| org.opentrafficsim.road.network.lane.object.trafficlight.SimpleTrafficLight | Line |
| --No comment-- | 102 |
| 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 |