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 | 111 | TODO, FIXME, XXX |
Each tag is detailed below:
Number of occurrences found in the code: 111
| org.opentrafficsim.graphs.ContourPlotTest | Line |
|---|---|
| looks wrong / PK | 427 |
| 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. | 267 |
| org.opentrafficsim.graphs.TrajectoryPlotTest | Line |
| adapt to new path (List<Lane>) concept | 35 |
| we take the first (and only) lane on which the vehicle is registered. | 174 |
| org.opentrafficsim.road.car.LaneBasedIndividualCar | Line |
| Should throw a more specific Exception type | 543 |
| org.opentrafficsim.road.car.LaneBasedTemplateCar | Line |
| check that none of the variables (except animationClass) is null, and throw an exception if it is. | 299 |
| org.opentrafficsim.road.gtu.AbstractLaneBasedGTUTest | Line |
| There should be a better way to check equality of RelativePosition | 164 |
| There should be a (better) way to check equality of RelativePosition | 270 |
| There should be a (better) way to check equality of RelativePosition | 306 |
| --No comment-- | 375 |
| --No comment-- | 377 |
| org.opentrafficsim.road.gtu.TemplateGTUTypeTest | Line |
| assertTrue("Passengers cars are allowed on a no trucks lane", passengerCar.isCompatible(trucksForbidden)); | 113 |
| assertFalse("Trucks are not allowed on a no trucks lane", truck.isCompatible(trucksForbidden)); | 114 |
| assertFalse("Passenger cars are not allowed on a trucks only lane", passengerCar.isCompatible(trucksOnly)); | 115 |
| assertTrue("Trucks are allowed on a trucks only lane", truck.isCompatible(trucksOnly)); | 116 |
| assertTrue("Passenger cars are allowed on an urban road", passengerCar.isCompatible(urbanRoad)); | 117 |
| assertTrue("Trucks are allowed on an urban road", truck.isCompatible(urbanRoad)); | 118 |
| assertFalse("Passenger cars are not allowed on a bicycle path", passengerCar.isCompatible(bicycleLane)); | 119 |
| assertFalse("Trucks are not allowed on an urban road", truck.isCompatible(bicycleLane)); | 120 |
| org.opentrafficsim.road.gtu.following.GTUFollowingModel | Line |
| We should probably add a <i>be ready to stop before</i> argument to prevent vehicles that cannot see their leader, or should slow down for a crossing from accelerating to unsafe speeds. | 24 |
| We should probably add a <i>be ready to stop before</i> argument to prevent vehicles that cannot see their leader, or should slow down for a crossing from accelerating to unsafe speeds. | 38 |
| We should probably add a <i>be ready to stop before</i> argument to prevent vehicles that cannot see their leader, or should slow down for a crossing from accelerating to unsafe speeds. | 59 |
| org.opentrafficsim.road.gtu.following.IDM | Line |
| compute logWeightedAccelerationTimes2 only once per run | 103 |
| org.opentrafficsim.road.gtu.generator.AbstractGTUGenerator | Line |
| 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())) | 319 |
| XX STUB dummy route | 320 |
| org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU | Line |
| 500? | 358 |
| 200? | 360 |
| Collecting information about nearby traffic should be done in a separate class. This is a very basic operation in OTS. | 373 |
| whether we drive on the right should be stored in some central place. | 391 |
| detect that a required lane change was blocked and, if it was, do something to find/create a gap. | 406 |
| Put the "update state to current time" code in a separate method and call that method at the start of this (move) method. | 415 |
| make lane changes gradual (not instantaneous; like now) | 443 |
| correct?? | 616 |
| if (lane.getSensors() contains SinkSensor => return LaneBasedRouteNavigator.NOLANECHANGENEEDED | 626 |
| correct?? | 688 |
| look if more lanes are entered in one timestep, and continue the algorithm with the remainder of the time... | 752 |
| this happens -- how can that be? | 758 |
| --No comment-- | 763 |
| look if more lanes are exited in one timestep, and continue the algorithm with the remainder of the time... | 781 |
| should be false? | 787 |
| Only follow links on the Route if there is a "real" Route | 1037 |
| use new functions of the Navigator if (this.getRoute() == null || this.getRoute().size() == 0 /* XXX STUB dummy route */ || (this.routeNavigator.getRoute().containsLink((Link) lane.getParentLink()))) | 1038 |
| STUB dummy route | 1039 |
| Only follow links on the Route if there is a Route if (this.getRoute() == null || this.getRoute().size() == 0) /* XXX STUB dummy route */ || this.routeNavigator.getRoute().containsLink((Link) lane.getParentLink())) | 1214 |
| STUB dummy route | 1215 |
| is this formula for parallel() okay? | 1358 |
| should it not be extended with several || clauses? | 1359 |
| org.opentrafficsim.road.gtu.lane.AbstractTrafficLight | Line |
| solve problem when point is still on previous lane. | 233 |
| org.opentrafficsim.road.gtu.lane.LaneBlock | Line |
| use animationClass | 127 |
| org.opentrafficsim.road.gtu.lane.LaneBlockOnOff | Line |
| --No comment-- | 52 |
| org.opentrafficsim.road.gtu.lane.changing.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 | 44 |
| org.opentrafficsim.road.gtu.lane.changing.LaneChangeModelTest | Line |
| create a LinkAnimation if the simulator is compatible with that. | 72 |
| The current LinkAnimation is too bad to use... | 73 |
| Decide what type of overtaking conditions we want in this test | 102 |
| test/prove the expected differences between Egoistic and Altruistic | 252 |
| prove that the most restrictive car in the other lane determines what happens | 253 |
| test merge into overtaking lane | 254 |
| org.opentrafficsim.road.network.LinkLocationTest | Line |
| distance to location on another link (not yet possible; currently ALWAYS returns null) | 63 |
| org.opentrafficsim.road.network.factory.XMLNetworkGeneratorTest | Line |
| repair headway in such a way that vehicle does not have to break (safe distance) | 122 |
| org.opentrafficsim.road.network.factory.opendrive.JunctionTag | Line |
| parse junction.connection | 53 |
| parse junction.laneLink | 54 |
| parse junction.priority | 55 |
| parse junction.controller | 56 |
| org.opentrafficsim.road.network.factory.opendrive.OpenDriveNetworkLaneParser | Line |
| parse header, geoReference and check version number of OpenDrive | 109 |
| org.opentrafficsim.road.network.factory.osm.OSMNetwork | Line |
| should probably assign all link end points of toNode to fromNode | 232 |
| Network looks 'crooked' after using this. This function checks for and removes redundancies between the networks links. | 273 |
| PK thinks that this method could remove a junction from a through-road | 295 |
| org.opentrafficsim.road.network.factory.osm.output.Convert | Line |
| How to figure out whether to keep left, right or keep lane? | 131 |
| how does the caller deal with a null result? (Answer: not!) | 195 |
| This is the German standard car lane width | 519 |
| German default bikepath width | 547 |
| German default footpath width | 551 |
| the following code assumes right-hand-side driving. | 619 |
| overtaking conditions per country and/or type of road? | 623 |
| overtaking conditions per country and/or type of road? | 633 |
| overtaking conditions per country and/or type of road? | 641 |
| 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) | 710 |
| org.opentrafficsim.road.network.factory.xml.FillTag | Line |
| GTUColorer | 192 |
| create a FILL | 218 |
| GTUMix | 220 |
| RouteMix | 221 |
| ShortestRoute | 222 |
| ShortestRouteMix | 223 |
| org.opentrafficsim.road.network.factory.xml.GTUColorerTag | Line |
| use parameters for AccelerationGTUColorer | 80 |
| use parameters for LaneChangeUrgeGTUColorer | 91 |
| default colorer | 104 |
| org.opentrafficsim.road.network.factory.xml.GTUTag | Line |
| probably ok to generate a new model for each GTU 'type'. | 135 |
| probably ok to generate a new model for each GTU 'type'. | 154 |
| org.opentrafficsim.road.network.factory.xml.GeneratorTag | Line |
| GTUMix | 281 |
| RouteMix | 282 |
| ShortestRoute | 283 |
| ShortestRouteMix | 284 |
| org.opentrafficsim.road.network.factory.xml.GlobalTag | Line |
| add other GTUColorer tags | 49 |
| parse other GTUColorer tags | 103 |
| org.opentrafficsim.road.network.factory.xml.Links | Line |
| correct? | 441 |
| correct? | 458 |
| LISTGENERATOR | 584 |
| org.opentrafficsim.road.network.factory.xml.ListGeneratorTag | Line |
| GTUColorer | 130 |
| org.opentrafficsim.road.network.factory.xml.NodeTag | Line |
| slope as an angle. | 45 |
| slope for the Node. | 80 |
| org.opentrafficsim.road.network.factory.xml.XmlNetworkLaneParser | Line |
| shortestRoute, routeMix, ShortestRouteMix | 178 |
| org.opentrafficsim.road.network.lane.Lane | Line |
| 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. | 690 |
| Hack; do not sample dummy vehicle at lane drop | 829 |
| is this what we want, or should we throw exception? | 854 |
| org.opentrafficsim.road.network.lane.LaneTest | Line |
| what overtaking conditions do we ant to test in this unit test? | 79 |
| what overtaking conditions do we ant to test in this unit test? | 136 |
| what overtaking conditions do we ant to test in this unit test? | 153 |
| what overtaking conditions do we ant to test in this unit test? | 213 |
| org.opentrafficsim.road.network.lane.SensorTest | Line |
| check that the sensor is triggered at the right time. | 160 |
| org.opentrafficsim.road.network.route.AbstractLaneBasedRouteNavigator | Line |
| this method should probably be moved into the CrossSectionLink class | 82 |
| org.opentrafficsim.road.network.route.CompleteLaneBasedRouteNavigator | Line |
| If all but one of these have no Lane compatible with gtuType, dying here is a bit premature | 206 |
