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 | 194 | TODO, FIXME, XXX |
Each tag is detailed below:
Number of occurrences found in the code: 194
| org.opentrafficsim.graphs.ContourPlotTest | Line |
|---|---|
| @Test | 93 |
| @Test | 111 |
| @Test | 169 |
| @Test | 187 |
| looks wrong / PK | 504 |
| org.opentrafficsim.graphs.FundamentalDiagram | Line |
| --No comment-- | 621 |
| org.opentrafficsim.graphs.FundamentalDiagramPlotTest | Line |
| @Test | 79 |
| THIS TEST FAILS!!! assertEquals("Density should be " + expectedDensity, expectedDensity, value, 0.00001); | 178 |
| 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. | 333 |
| org.opentrafficsim.graphs.TrajectoryPlotTest | Line |
| we take the first (and only) lane on which the vehicle is registered. | 204 |
| org.opentrafficsim.road.DefaultTestParameters | Line |
| add another else to catch any unanticipated cases? | 78 |
| Explain why this exception can/should be ignored. do not set parameter without default value | 82 |
| 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. | 101 |
| 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()); | 139 |
| removeLane should throw an Error when the car is not on that lane (currently this is silently ignored) | 383 |
| figure out why the added lane has a non-zero position | 384 |
| org.opentrafficsim.road.gtu.LaneBasedGTUTest | Line |
| see how we can ask the vehicle to look this far ahead | 177 |
| see how we can ask the vehicle to look this far behind | 185 |
| see how we can ask the vehicle to look 'forwardMaxDistance' ahead | 219 |
| follower = truck.headway(reverseMaxDistance); | 240 |
| org.opentrafficsim.road.gtu.LaneBasedTemplateGTUTypeTest | Line |
| assertTrue("Passengers cars are allowed on a no trucks lane", passengerCar.isCompatible(trucksForbidden)); | 248 |
| assertFalse("Trucks are not allowed on a no trucks lane", truck.isCompatible(trucksForbidden)); | 249 |
| assertFalse("Passenger cars are not allowed on a trucks only lane", passengerCar.isCompatible(trucksOnly)); | 250 |
| assertTrue("Trucks are allowed on a trucks only lane", truck.isCompatible(trucksOnly)); | 251 |
| assertTrue("Passenger cars are allowed on an urban road", passengerCar.isCompatible(urbanRoad)); | 252 |
| assertTrue("Trucks are allowed on an urban road", truck.isCompatible(urbanRoad)); | 253 |
| assertFalse("Passenger cars are not allowed on a bicycle path", passengerCar.isCompatible(bicycleLane)); | 254 |
| assertFalse("Trucks are not allowed on an urban road", truck.isCompatible(bicycleLane)); | 255 |
| org.opentrafficsim.road.gtu.animation.DefaultSwitchableGTUColorer | Line |
| same class name as in core, w\o additional functionality, while the core one is used in AbstractWrappableAnimation | 24 |
| org.opentrafficsim.road.gtu.generator.AbstractGTUGenerator | Line |
| different from this.direction? | 276 |
| THIS METHOD IS ALSO IN PERCEPTION -- DON'T DUPLICATE; ALSO, THIS VERSION IS WRONG. | 337 |
| 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())) | 360 |
| XX STUB dummy route | 361 |
| THIS METHOD IS ALSO IN PERCEPTION -- DON'T DUPLICATE; ALSO, THIS VERSION IS WRONG. | 397 |
| 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())) | 420 |
| XX STUB dummy route | 421 |
| org.opentrafficsim.road.gtu.generator.od.ODApplier | Line |
| sinks? white extension links? | 133 |
| clean up stream acquiring code after task OTS-315 has been completed | 138 |
| org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU | Line |
| should be indicated with a Parameter | 150 |
| can be removed after testing period | 154 |
| can be removed after testing period | 158 |
| When requesting the position at the end of the plan, which will be on a further lane, this lane is not yet part of the lanes in the current operational plan. This can be upstream or downstream depending on the direction of travel. We might check whether getDirection(lane)=DIR_PLUS and upstream=false, or getDirection(lane)=DIR_MINUS and upstream=true, to then use LaneDirection.getNextLaneDirection(this) to obtain the next lane. This is only required if nextLanes originally had more than 1 lane, otherwise we can simply use that one lane. Problem is that the search might go on far or even eternally (on a circular network), as projection simply keeps failing because the GTU is actually towards the other longitudinal direction. Hence, the heuristic used before this method is called should change and first always search against the direction of travel, and only consider lanes in currentLanes, while the consecutive search in the direction of travel should then always find a point. We could build in a counter to prevent a hanging software. | 985 |
| widest lane in case we are registered on more than one lane with the reference point? | 1032 |
| lane that leads to our location or not if we are registered on parallel lanes? | 1033 |
| org.opentrafficsim.road.gtu.lane.LaneBasedIndividualGTU | Line |
| Should throw a more specific Exception type | 543 |
| 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 | 125 |
| 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.AbstractPerceptionIterable | Line |
| this let's us ignore an object that is registered on the next lane, but who's tail may be on this lane | 237 |
| org.opentrafficsim.road.gtu.lane.perception.RollingLaneStructure | Line |
| on complex networks, e.g. with sections connectors where lane changes are not possible, the update may fail | 224 |
| we now assume everything is on the route, but merges could be ok without route so, without a route we should be able to recognize which upstream 'nextLane' is on the other link | 1014 |
| org.opentrafficsim.road.gtu.lane.perception.RollingLaneStructureRecord | Line |
| clear on network change, with an event listener? | 45 |
| org.opentrafficsim.road.gtu.lane.perception.categories.AbstractDelayedNeighborsPerception | Line |
| The reaction time may differ between observed objects and vary over time | 113 |
| private when DSOL allows | 241 |
| 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. | 253 |
| org.opentrafficsim.road.gtu.lane.perception.categories.DirectDefaultSimplePerception | Line |
| end of lanepath | 637 |
| allow observation of other objects as well. | 843 |
| --No comment-- | 939 |
| calculate real overlaps | 955 |
| expand for other types of Headways result.add(new HeadwayGTUSimple(p.getId(), ((AbstractHeadwayGTU) p).getGtuType(), new Length(Double.NaN, LengthUnit.SI), p.getLength(), p.getSpeed(), p.getAcceleration())); | 1013 |
| org.opentrafficsim.road.gtu.lane.perception.categories.DirectInfrastructurePerception | Line |
| splits | 450 |
| splits | 461 |
| org.opentrafficsim.road.gtu.lane.perception.categories.DirectIntersectionPerception | Line |
| if conflicts span multiple lanes, this within-lane search fails | 163 |
| get from link combination (needs to be a map property on the links) | 193 |
| limit 'conflictingVisibility' to first upstream traffic light, so GTU's behind it are ignored | 210 |
| stop lines (current models happen not to use this, but should be possible) | 221 |
| --No comment-- | 249 |
| org.opentrafficsim.road.gtu.lane.perception.categories.DirectNeighborsPerception | Line |
| not use this check when synchronizing or cooperating | 508 |
| org.opentrafficsim.road.gtu.lane.perception.categories.HistoricalNeighborsPerception | Line |
| how to get a specific String to getCrossSection??? | 86 |
| org.opentrafficsim.road.gtu.lane.perception.headway.HeadwayGTUReal | Line |
| --No comment-- | 261 |
| --No comment-- | 290 |
| org.opentrafficsim.road.gtu.lane.perception.headway.HeadwayGTUType | Line |
| create SpeedLimitInfo on the basis of this.gtuTypeAssumptions.getLaneTypeMaxSpeed(...) | 152 |
| org.opentrafficsim.road.gtu.lane.plan.operational.LaneChange | Line |
| get rid of cast to AbstractLaneBasedGTU | 282 |
| org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder | Line |
| driving with negative speeds (backward driving) is not yet supported. <p> Copyright (c) 2013-2018 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> | 42 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedCFLCTacticalPlanner | Line |
| how to handle objects on this lane or another lane??? | 137 |
| these two info's are not used | 152 |
| skip if: - we are in the right lane and drive at max speed or we accelerate maximally - there are no other lanes | 157 |
| whether we drive on the right should be stored in some central place. | 163 |
| 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. | 239 |
| these comparisons to -10 is ridiculous. | 295 |
| move suitability to LanePerception? Return the suitability for the current lane, left adjacent lane or right adjacent lane. | 385 |
| this code assumes GTUDirectionality.DIR_PLUS on all lanes of interest | 440 |
| --No comment-- | 619 |
| this method should probably be moved into the CrossSectionLink class | 677 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedGTUFollowingDirectedChangeTacticalPlanner | Line |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 297 |
| Object & GTU | 298 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 344 |
| GTU & Object | 345 |
| remove this hack | 512 |
| should it be getObjectType().isGtu() or !getObjectType().isDistanceOnly() ? | 559 |
| Object & GTU | 560 |
| make type of plan (Egoistic, Altruistic) parameter of the class | 570 |
| make the elasticities 2.0 and 0.1 parameters of the class | 572 |
| 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... | 86 |
| org.opentrafficsim.road.gtu.lane.tactical.LaneBasedTacticalPlanner | Line |
| private when we use java 9 | 87 |
| private when we use java 9 | 118 |
| org.opentrafficsim.road.gtu.lane.tactical.following.AbstractGTUFollowingModelMobil | Line |
| Under which circumstances can getDistance() be NULL? Should that indeed result in TOODANGEROUS? | 62 |
| org.opentrafficsim.road.gtu.lane.tactical.following.FixedAccelerationModel | Line |
| should be specified in constructor | 89 |
| org.opentrafficsim.road.gtu.lane.tactical.following.IDMOld | Line |
| maxDistance dV is the approach speed | 113 |
| org.opentrafficsim.road.gtu.lane.tactical.following.IDMPlusOld | Line |
| maxDistance | 119 |
| org.opentrafficsim.road.gtu.lane.tactical.following.SequentialFixedAccelerationModel | Line |
| incorporate stepSize | 157 |
| 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 | 68 |
| 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-2018 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.AccelerationBusStop | Line |
| this is much more complex: tail blocking other traffic? other bus in front? many people at bus stop? | 40 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.AccelerationConflicts | Line |
| consider adjacent lanes before and during lane change | 53 |
| this is too simple, needs to be consistent with gap-acceptance or GTU's may not change | 80 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.AccelerationNoRightOvertake | Line |
| ignore incentive if we need to change lane for the route | 50 |
| only sensible if the left leader can change right; add this info to HeadwayGTU? | 65 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.IncentiveCourtesy | Line |
| factor -a/b as influence factor is heavy in calculation, consider v<vEgo & 1-s/x0 | 87 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.IncentiveSocioSpeed | Line |
| keep left or right rules | 41 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRS | Line |
| make this automatic within the perception itself, e.g. by a lane change event from the tactical planner | 217 |
| org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo | Line |
| vehicle not ahead and not backwards but completely adjacent | 118 |
| infinite -> some limited space, speed also one leader | 351 |
| definition of 'right most lane' does not account for ramps and weaving sections | 639 |
| org.opentrafficsim.road.gtu.lane.tactical.toledo.ToledoCarFollowing | Line |
| speed difference with reaction time | 146 |
| org.opentrafficsim.road.gtu.lane.tactical.toledo.ToledoPerception | Line |
| updateInfrastructureLaneChangeInfo with split number | 33 |
| org.opentrafficsim.road.gtu.lane.tactical.util.ConflictUtil | Line |
| do not ignore vehicles upstream of conflict if they have green | 63 |
| conflict over multiple lanes (longitudinal in own direction) | 64 |
| a) yielding while having priority happens only when leaders is standing still on conflict (then its useless...) b) two vehicles can remain upstream of merge if vehicle stands on merge but leaves some space to move probably 1 is yielding, and 1 is courtesy yielding as the other stands still c) they might start moving together and collide further down (no response to negative headway on merge) | 65 |
| depending on left/right-hand traffic | 196 |
| depending on rules, we may need to stop and not just yield | 210 |
| --No comment-- | 264 |
| what to do when we happen to be in the stopping distance? Stopping might be reasonable, while car-following might give strong deceleration due to s < s0. | 274 |
| only within visibility | 431 |
| safety factor? | 481 |
| conflicting vehicle on crossing conflict, but will leave sooner then we enter, so no problem? | 577 |
| more generally, also upstream conflicting vehicles at crossings may leave the conflict before we enter | 578 |
| stopping | 766 |
| all-stop behavior | 779 |
| org.opentrafficsim.road.gtu.lane.tactical.util.lmrs.GapAcceptance | Line |
| --No comment-- | 49 |
| org.opentrafficsim.road.gtu.lane.tactical.util.lmrs.Synchronization | Line |
| replace this hack with something that properly accounts for overshoot this method also introduces very strong deceleration at low speeds, as the time step makes bMin go from 3.4 (ignored, so maybe 1.25 acceleration applied) to >10 | 83 |
| find a better solution for this inefficient hack... when to ignore a vehicle for synchronization? | 440 |
| org.opentrafficsim.road.gtu.perception.VerifyPerceptionCategoryMethods | Line |
| fail... | 215 |
| fail... | 221 |
| org.opentrafficsim.road.gtu.strategical.LaneBasedStrategicalRoutePlannerTest | Line |
| Gtu cannot be null anymore... LaneBasedStrategicalRoutePlanner lbsrp = new LaneBasedStrategicalRoutePlanner(params, tacticalPlanner, null); | 55 |
| org.opentrafficsim.road.gtu.strategical.od.SplitFraction | Line |
| let interpolation interpolate itself | 175 |
| org.opentrafficsim.road.gtu.strategical.route.LaneBasedStrategicalRoutePlanner | Line |
| Is there a reason not to iterate over csLink.getLanes()? | 233 |
| this takes the first in the set of links that connects the correct nodes; does not handle parallel links consistently | 283 |
| org.opentrafficsim.road.network.LinkLocationTest | Line |
| distance to location on another link (not yet possible; currently ALWAYS returns null) | 77 |
| org.opentrafficsim.road.network.animation.ConflictAnimation | Line |
| should ConflictAnimation implement the ClonableRenderable2DInterface? | 33 |
| org.opentrafficsim.road.network.animation.StripeAnimation | Line |
| startOffset does not work if a dash falls inside of it (so below the offset is 2.99m, rather than 3m) | 64 |
| org.opentrafficsim.road.network.lane.CrossSectionLink | Line |
| per GTUDirectionality / LongitudinalDirectionality? | 46 |
| work in a different way with lane numbers to align to standard lane numbering. | 55 |
| allow for the removal of a Lane; currently this is not possible. | 62 |
| org.opentrafficsim.road.network.lane.CurveTest | Line |
| finish writing this test | 105 |
| org.opentrafficsim.road.network.lane.Lane | Line |
| allow for direction-dependent speed limit | 111 |
| allow for direction-dependent sensors | 121 |
| allow for direction-dependent lane objects | 128 |
| allow for direction-dependent overtaking conditions | 181 |
| constructor calls with this(...) | 412 |
| take the cross section slices into account... | 473 |
| take the cross section slices into account... | 489 |
| take the cross section slices into account... | 513 |
| take the cross section slices into account... | 529 |
| this is a hack for when sink sensors aren't perfectly adjacent or the GTU overshoots with nose due to curvature | 785 |
| 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. | 1272 |
| this should return something immutable | 1292 |
| determine if this should synchronize on this.nextLanes | 1301 |
| And is it aligned with its next lane? | 1321 |
| And is it aligned with its next lane? | 1336 |
| this should return something immutable | 1369 |
| And is it aligned with its next lane? | 1397 |
| And is it aligned with its next lane? | 1412 |
| check that the directionality of this Lane does not conflict with that of the parent the OTSLink for (GTUType gtuType : this.directionalityMap.keySet()) { LongitudinalDirectionality directionality = this.directionalityMap.get(gtuType); if (!getParentLink().getDirectionality(gtuType).contains(directionality)) { throw new NetworkException("Lane " + toString() + " allows " + gtuType + " a directionality of " + directionality + " which is not present in the overarching link " + getParentLink().toString()); } } | 1683 |
| let HistoricalArrayList return an Immutable (WRAP) of itself | 1700 |
| org.opentrafficsim.road.network.lane.LaneTest | Line |
| what overtaking conditions do we want to test in this unit test? | 84 |
| This test for expectedLateralCenterOffset fails | 100 |
| what overtaking conditions do we ant to test in this unit test? | 138 |
| what overtaking conditions do we ant to test in this unit test? | 153 |
| what overtaking conditions do we want to test in this unit test? | 212 |
| org.opentrafficsim.road.network.lane.SensorTest | Line |
| this is not true anymore with OperationalPlans, Perception, etc => | 142 |
| the number of events that should be scheduled can vary per models chosen | 143 |
| assertEquals("There should be three scheduled events (trigger, leaveLane, | 144 |
| 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. | 145 |
| assertEquals("Trigger event should be around 38.3403", 38.3403, | 149 |
| triggerEvent.getAbsoluteExecutionTime().get().getSI(), 0.0001); | 150 |
| check that the sensor is triggered at the right time. | 178 |
| org.opentrafficsim.road.network.lane.changing.OvertakingConditions | Line |
| All these "Right/LeftSet" classes should probably use Compatibility instead of full sets. <p> Copyright (c) 2013-2018 Delft University of Technology, PO Box 5, 2600 AA, Delft, the Netherlands. All rights reserved. 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 Sep 13, 2015 | 361 |
| org.opentrafficsim.road.network.lane.conflict.Conflict | Line |
| why not create a getter for the gtu in the super class? <p> Copyright (c) 2013-2018 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/node/13">OpenTrafficSim License</a>. <p> | 476 |
| org.opentrafficsim.road.network.lane.conflict.ConflictBuilder | Line |
| use z-coordinate for intersections of lines | 38 |
| org.opentrafficsim.road.network.lane.object.sensor.DirectionalOccupancySensor | Line |
| enforce clone method | 63 |
| org.opentrafficsim.road.network.lane.object.sensor.NonDirectionalOccupancySensor | Line |
| enforce clone method | 66 |
| org.opentrafficsim.road.network.lane.object.sensor.TrafficLightSensor | Line |
| Possibly provide the GTUTypes that trigger the sensor as an argument for the constructor | 90 |
| not handling backwards driving GTU | 346 |
| should the parent of the clone be our parent??? And should the (cloned) parent not construct its own flank sensors? | 639 |
| org.opentrafficsim.road.network.lane.object.sensor.TrafficLightSensorTest | Line |
| @Test | 199 |
| THIS TEST FAILS!! assertEquals("event list should contain one event (due to creation of the GTU on the detector)", 1, this.loggedEvents.size()); | 283 |
| org.opentrafficsim.road.network.sampling.RoadSampler | Line |
| GTUDirectionality from Lane.GTU_ADD_EVENT | 220 |
| Length from Lane.GTU_ADD_EVENT | 232 |
| GTUDirectionality from Lane.GTU_ADD_EVENT | 261 |
| org.opentrafficsim.road.network.speed.package-info | Line |
| --No comment-- | 21 |