Tag List Report

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 230 TODO, FIXME, XXX

Each tag is detailed below:

Open issues in the code

Number of occurrences found in the code: 230

org.opentrafficsim.graphs.ContourPlotTest Line
looks wrong / PK 512
org.opentrafficsim.graphs.FundamentalDiagram Line
--No comment-- 617
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. 271
lane change causes error... 532
org.opentrafficsim.graphs.TrajectoryPlotTest Line
adapt to new path (List<Lane>) concept 37
we take the first (and only) lane on which the vehicle is registered. 176
org.opentrafficsim.road.gtu.AbstractLaneBasedGTUTest Line
removeLane should throw an Error when the car is not on that lane (currently this is silently ignored) 387
figure out why the added lane has a non-zero position 388
org.opentrafficsim.road.gtu.LaneBasedGTUTest Line
see how we can ask the vehicle to look this far ahead 175
see how we can ask the vehicle to look this far behind 182
see how we can ask the vehicle to look 'forwardMaxDistance' ahead 213
follower = truck.headway(reverseMaxDistance); 234
org.opentrafficsim.road.gtu.LaneBasedTemplateGTUTypeTest Line
assertTrue("Passengers cars are allowed on a no trucks lane", passengerCar.isCompatible(trucksForbidden)); 244
assertFalse("Trucks are not allowed on a no trucks lane", truck.isCompatible(trucksForbidden)); 245
assertFalse("Passenger cars are not allowed on a trucks only lane", passengerCar.isCompatible(trucksOnly)); 246
assertTrue("Trucks are allowed on a trucks only lane", truck.isCompatible(trucksOnly)); 247
assertTrue("Passenger cars are allowed on an urban road", passengerCar.isCompatible(urbanRoad)); 248
assertTrue("Trucks are allowed on an urban road", truck.isCompatible(urbanRoad)); 249
assertFalse("Passenger cars are not allowed on a bicycle path", passengerCar.isCompatible(bicycleLane)); 250
assertFalse("Trucks are not allowed on an urban road", truck.isCompatible(bicycleLane)); 251
org.opentrafficsim.road.gtu.animation.DefaultSwitchableGTUColorer Line
LaneChangeUrgeGTUColorer COLORERS[3] = new LaneChangeUrgeGTUColorer(new Length.Rel(10, LengthUnit.METER), new Length.Rel(1000, LengthUnit.METER)); 38
org.opentrafficsim.road.gtu.generator.AbstractGTUGenerator Line
same instance? clone? 196
same instance? clone? 197
different from this.direction? 247
THIS METHOD IS ALSO IN PERCEPTION -- DON'T DUPLICATE; ALSO, THIS VERSION IS RONG. 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. 371
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())) 395
XX STUB dummy route 396
org.opentrafficsim.road.gtu.generator.ListGTUGenerator Line
use given position in the constructor? 172
use given directionality in the constructor? 175
org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU Line
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? 602
look if more lanes are exited in one timestep, and continue the algorithm with the remainder of the time... 624
XXXXXXXXXXXXXXX Minus one ULP is not safe if you want to add the current time 641
XXXXXXXXXXXXXXX Should compute the time time at which the rear of the GTU exits the lane??? getSimulator().scheduleEventRel(new Time.Rel(timestep - Math.ulp(timestep), TimeUnit.SI), this, this, "leaveLane", new Object[] { lane, new Boolean(true) }); // TODO should be false? 642
should be false? 644
should be false? 661
direction dependent... <br> 668
Hack - this should be done more considerate -- fails at loops... 705
direction dependent... <br> 730
Hack - this should be done more considerate -- fails at loops... 767
org.opentrafficsim.road.gtu.lane.CopyOfAbstractLaneBasedGTU Line
WRONG - should be based on timeAtPosition() as the plan can have acc/dec/const segments 500
extra argument for DIR_MINUS driving direction? 514
look if more lanes are entered in one timestep, and continue the algorithm with the remainder of the time... 521
extra argument for DIR_MINUS driving direction? 553
extra argument for DIR_MINUS driving direction? 591
look if more lanes are exited in one timestep, and continue the algorithm with the remainder of the time... 613
XXXXXXXXXXXXXXX Minus one ULP is not safe if you want to add the current time 630
XXXXXXXXXXXXXXX Should compute the time time at which the rear of the GTU exits the lane??? getSimulator().scheduleEventRel(new Time.Rel(timestep - Math.ulp(timestep), TimeUnit.SI), this, this, "leaveLane", new Object[] { lane, new Boolean(true) }); // TODO should be false? 631
should be false? 633
should be false? 650
direction dependent... <br> 657
Hack - this should be done more considerate -- fails at loops... 694
direction dependent... <br> 719
Hack - this should be done more considerate -- fails at loops... 756
org.opentrafficsim.road.gtu.lane.LaneBasedGTU Line
change into LanePerception interface! 34
org.opentrafficsim.road.gtu.lane.LaneBasedIndividualGTU Line
Should throw a more specific Exception type 520
org.opentrafficsim.road.gtu.lane.changing.LaneChangeModelTest Line
create a LinkAnimation if the simulator is compatible with that. 88
The current LinkAnimation is too bad to use... 89
Decide what type of overtaking conditions we want in this test 119
test/prove the expected differences between Egoistic and Altruistic 282
prove that the most restrictive car in the other lane determines what happens 283
test merge into overtaking lane 284
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
private LaneStructure laneStructure; 87
end of lanepath 371
In other places in OTS LEFT is positive (and RIGHT is negative). This should be made more consistent. 641
getPerceivedObjects() in LanePerception 865
getPerceivedObjects() in LanePerception 936
org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder Line
driving with negative speeds (backward driving) is not yet supported. <p> Copyright (c) 2013-2015 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> 27
org.opentrafficsim.road.gtu.lane.tactical.AbstractLaneBasedTacticalPlanner Line
widest lane in case we are registered on more than one lane with the reference point 69
lane that leads to our location or not if we are registered on parallel lanes? 70
lane closest to length or 0 75
this method can probably disappear (lots of duplicated code) 284
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 143
whether we drive on the right should be stored in some central place. 149
Make a 100% lateral move from standing still... 241
make a gradual lateral move 245
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. 294
these comparisons to -10 is ridiculous. 347
correct?? 420
move suitability to LanePerception? Return the suitability for the current lane, left adjacent lane or right adjacent lane. 454
correct?? These nested if statements can be combined, but that would reduce readability of the code 493
this method should probably be moved into the CrossSectionLink class 791
org.opentrafficsim.road.gtu.lane.tactical.LaneBasedGTUFollowingChange0TacticalPlanner Line
make direction dependent! 149
we are driving... 162
we are driving... 208
make type of plan (Egoistic, Altruistic) parameter of the class 400
make the elasticities 2.0 and 0.1 parameters of the class 402
org.opentrafficsim.road.gtu.lane.tactical.LaneBasedGTUFollowingLaneChangeTacticalPlanner Line
we are driving... 175
we are driving... 217
if we move from standstill, create a longer plan, e.g. 4-5 seconds, with high acceleration! 391
make type of plan (Egoistic, Altruistic) parameter of the class 392
make the elasticities 2.0 and 0.1 parameters of the class 394
take the widest (now a random one) 407
if the distance is too small, do not build a path. Minimum = 0.5 * vehicle length 422
this should be solved in the time domain, not in the distance domain... 423
arbitrary... 424
arbitrary... 455
org.opentrafficsim.road.gtu.lane.tactical.LaneBasedGTUFollowingLaneChangeTacticalPlanner2 Line
multiple lanes!!!!!! 157
book out of not covered lanes... 177
book out of not covered lanes... 194
we are driving... 252
we are driving... 294
if we move from standstill, create a longer plan, e.g. 4-5 seconds, with high acceleration! 468
make type of plan (Egoistic, Altruistic) parameter of the class 469
make the elasticities 2.0 and 0.1 parameters of the class 471
take the widest (now a random one) 484
if the distance is too small, do not build a path. Minimum = 0.5 * vehicle length 499
this should be solved in the time domain, not in the distance domain... 500
arbitrary... 501
arbitrary... 532
org.opentrafficsim.road.gtu.lane.tactical.following.AbstractCarFollowingModel Line
do not implement GTUFollowingModelOld 24
Remove methods below 88
Auto-generated method stub 94
Auto-generated method stub 103
Auto-generated method stub 111
Auto-generated method stub 119
Auto-generated method stub 126
Auto-generated method stub 134
Auto-generated method stub 142
Auto-generated method stub 151
Auto-generated method stub 159
Auto-generated method stub 168
Auto-generated method stub 176
Auto-generated method stub 182
Auto-generated method stub 188
org.opentrafficsim.road.gtu.lane.tactical.following.FixedAccelerationModel Line
should be specified in constructor 79
org.opentrafficsim.road.gtu.lane.tactical.following.IDMOld Line
maxDistance dV is the approach speed 100
org.opentrafficsim.road.gtu.lane.tactical.following.IDMPlusOld Line
maxDistance 107
org.opentrafficsim.road.gtu.lane.tactical.following.SequentialFixedAccelerationModel Line
incorporate stepSize 146
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 56
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-2015 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> 13
org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRS Line
Remove this when other todo's are done, it is used as a placeholder where some acceleration needs to be determined. 117
Throw ParameterException 132
if changing lane (rand to disable annoying warnings) 150
Build the operational plan using minimum acceleration LaneOperationalPlanBuilder 161
relaxation 170
get neighboring vehicles, use their (perceived) car-following model with altered headway 222
get from perception / infrastructure, with relaxation 225
get from perception / infrastructure, with relaxation 230
switch on indicators (and off)? 239
get neighboring vehicles, use car-following model with altered headway 264
get neighboring vehicles, their indicators, use car-following model with altered headway 273
Build the operational plan using minimum acceleration and including a possible lane change using changeLeft/changeRight LaneOperationalPlanBuilder 286
adjust desired headway based on desire set T 294
speed limit 302
follower != self 303
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.TestOpenDriveParserNASA Line
parse these from the xodr-file. double latCenter = 37.40897623275873, lonCenter = -122.0246091728831;//sunnyvale double latCenter = 37.419933552777, lonCenter = -122.05752616111000;//nasa 216
org.opentrafficsim.road.network.factory.TestOpenDriveParserNoRTINew Line
dirty hack... 470
org.opentrafficsim.road.network.factory.TestOpenDriveParserSV Line
parse these from the xodr-file. double latCenter = 37.40897623275873, lonCenter = -122.0246091728831;//sunnyvale double latCenter = 37.419933552777, lonCenter = -122.05752616111000;//nasa 218
org.opentrafficsim.road.network.factory.XMLNetworkGeneratorTest Line
repair headway in such a way that vehicle does not have to brake (safe distance) 124
org.opentrafficsim.road.network.factory.opendrive.Controller Line
--No comment-- 63
org.opentrafficsim.road.network.factory.osm.OSMNetwork Line
should probably assign all link end points of toNode to fromNode 233
Network looks 'crooked' after using this. This function checks for and removes redundancies between the networks links. 274
PK thinks that this method could remove a junction from a through-road 296
org.opentrafficsim.road.network.factory.osm.output.Convert Line
How to figure out whether to keep left, right or keep lane? 136
How to figure out if this is a lane in one or two directions? For now, two is assumed... 137
how does the caller deal with a null result? (Answer: not!) 203
This is the German standard car lane width 546
German default bikepath width 574
German default footpath width 578
the following code assumes right-hand-side driving. 645
overtaking conditions per country and/or type of road? 649
overtaking conditions per country and/or type of road? 660
overtaking conditions per country and/or type of road? 669
test performance (memory- and time-wise) when the counters are replaced by ArrayList<OSMLink> which would obviate the need to do full searches over all Links to find OSMNodes that are source or sink. Reset the counters (should not be necessary unless this method is called more than once) 739
org.opentrafficsim.road.network.factory.vissim.VissimANMNetworkLaneParser Line
shortestRoute, routeMix, ShortestRouteMix 179
Make routes GTU specific. See what to do with GTUType.ALL for routes network.addRoute(GTUType.ALL, routeTag.route); } 203
org.opentrafficsim.road.network.factory.xml.FillTag Line
GTUColorer 194
create a FILL 221
GTUMix 223
RouteMix 224
ShortestRoute 225
ShortestRouteMix 226
org.opentrafficsim.road.network.factory.xml.GTUColorerTag Line
use parameters for AccelerationGTUColorer 79
default colorer 93
org.opentrafficsim.road.network.factory.xml.GTUTag Line
probably ok to generate a new model for each GTU 'type'. 136
probably ok to generate a new model for each GTU 'type'. 155
org.opentrafficsim.road.network.factory.xml.GeneratorTag Line
parse direction 144
GTUMix 306
RouteMix 307
ShortestRoute 308
ShortestRouteMix 309
org.opentrafficsim.road.network.factory.xml.GlobalTag Line
add other GTUColorer tags 50
parse other GTUColorer tags 104
org.opentrafficsim.road.network.factory.xml.Links Line
test for over-specification (i.e. node direction was already specified) 222
move the radius if there is an start and end offset? How? 409
Map<GTUType, LongitudinalDirectionality> linkDirections = new HashMap<>(); 491
correct? 540
correct? 557
LISTGENERATOR 707
org.opentrafficsim.road.network.factory.xml.ListGeneratorTag Line
GTUColorer 130
org.opentrafficsim.road.network.factory.xml.NodeTag Line
slope as an angle. 44
slope for the Node. 80
org.opentrafficsim.road.network.factory.xml.XmlNetworkLaneParser Line
shortestRoute, routeMix, ShortestRouteMix 184
Make routes GTU specific. See what to do with GTUType.ALL for routes 207
org.opentrafficsim.road.network.lane.Lane Line
allow for direction-dependent speed limit 90
allow for direction-dependent sensors 97
allow for direction-dependent overtaking conditions 133
take the cross section slices into account... 387
take the cross section slices into account... 401
take the cross section slices into account... 424
take the cross section slices into account... 438
make sure triggering is done right when driving in DIR_MINUS direction 631
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. 815
And is it aligned with its next lane? 861
And is it aligned with its next lane? 875
And is it aligned with its next lane? 933
And is it aligned with its next lane? 947
org.opentrafficsim.road.network.lane.LaneTest Line
what overtaking conditions do we ant to test in this unit test? 80
what overtaking conditions do we ant to test in this unit test? 137
what overtaking conditions do we ant to test in this unit test? 154
what overtaking conditions do we ant to test in this unit test? 215
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
not triggered in next half second. 148
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. 177