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 | 92 | TODO, FIXME, XXX |
Each tag is detailed below:
Open issues in the code
Number of occurrences found in the code: 92
| org.opentrafficsim.road.DefaultTestParameters | Line |
|---|---|
| add another else to catch any unanticipated cases? | 80 |
| Explain why this exception can/should be ignored. do not set parameter without default value | 84 |
| 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. | 90 |
| 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()); | 138 |
| removeLane should throw an Error when the car is not on that lane (currently this is silently ignored) | 293 |
| figure out why the added lane has a non-zero position | 294 |
| org.opentrafficsim.road.gtu.LaneBasedGtuTest | Line |
| see how we can ask the vehicle to look this far ahead | 179 |
| see how we can ask the vehicle to look this far behind | 186 |
| see how we can ask the vehicle to look 'forwardMaxDistance' ahead | 219 |
| follower = truck.headway(reverseMaxDistance); | 240 |
| org.opentrafficsim.road.gtu.generator.Platoons | Line |
| end time of platoon may be in next demand period, which makes the demand non-linear | 311 |
| org.opentrafficsim.road.gtu.lane.LaneBasedGtu | Line |
| move this to super.init(...), and remove setOperationalPlan(...) method Give the GTU a 1 micrometer long operational plan, or a stand-still plan, so the first move and events will work | 265 |
| instead of getNetwork().getNodeMap().values(), using spatial tree would be a good alternative perhaps even a findClosest() method. | 554 |
| org.opentrafficsim.road.gtu.lane.control.DelayedActuation | Line |
| numerical implementation of tau rule | 43 |
| org.opentrafficsim.road.gtu.lane.perception.categories.DirectInfrastructurePerception | Line |
| this is very limited information regarding what the prospect could have, is this is only maximum vehicle speed, and legal speed on the lane | 154 |
| org.opentrafficsim.road.gtu.lane.perception.mental.channel.IntersectionPerceptionChannel | Line |
| visibility | 136 |
| org.opentrafficsim.road.gtu.lane.perception.object.PerceivedConflict | Line |
| stop lines (current models happen not to use this, but should be possible) | 154 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.AccelerationBusStop | Line |
| this process is much more complex: tail blocking other traffic? other bus in front? many people at bus stop? | 43 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.AccelerationNoRightOvertake | Line |
| ignore incentive if we need to change lane for the route | 67 |
| only sensible if the left leader can change right; add this info to HeadwayGtu? | 82 |
| 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 | 96 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.IncentiveQueue | Line |
| a ramp-metering traffic light triggers this incentive with possible cooperation from the main line | 79 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.IncentiveSocioSpeed | Line |
| keep left or right rules | 41 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.IncentiveSpeed | Line |
| SpeedWithCourtesy now uses TrafficPerception, which embeds the courtesy part. How to do this? | 49 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.Lmrs | Line |
| consider route in incentives (only if not on current lane?) | 114 |
| org.opentrafficsim.road.gtu.lane.tactical.lmrs.LmrsFactory | Line |
| parameters of enhanced intersection model | 813 |
| update conflict channel task to better intersection/infrastructure based model | 977 |
| org.opentrafficsim.road.gtu.lane.tactical.util.ConflictUtil | Line |
| do not ignore vehicles upstream of conflict if they have green | 72 |
| conflict over multiple lanes (longitudinal in own direction) | 73 |
| 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) | 74 |
| priority rules of busses should be handled differently, this also makes a GTU type unnecessary here | 201 |
| depending on left/right-hand traffic | 211 |
| safety factor? | 547 |
| this check is simplistic, designed quick and dirty, it just adds 3s as a safe gap | 593 |
| stopping | 808 |
| all-stop behavior | 821 |
| org.opentrafficsim.road.gtu.lane.tactical.util.lmrs.Cooperation | Line |
| only cooperate if merger still quite fast or there's congestion downstream anyway (which we can better estimate than only considering the direct leader | 99 |
| org.opentrafficsim.road.gtu.lane.tactical.util.lmrs.GapAcceptance | Line |
| --No comment-- | 60 |
| org.opentrafficsim.road.gtu.lane.tactical.util.lmrs.LmrsUtil | Line |
| this approach is a hack | 494 |
| org.opentrafficsim.road.gtu.perception.RelativeLaneTest | Line |
| Wait for Wouter to indicate whether numLanes == 0 is really permitted when lat != LateralDirectionality.NONE | 77 |
| fix if 0 is only permitted for LateralDirectionality.NONE | 81 |
| org.opentrafficsim.road.gtu.perception.VerifyPerceptionCategoryMethods | Line |
| to what extent do we want to prescribe this now that we have more flexible perception categories | 74 |
| fail("Class " + c + " does not have a field '" + field + "*', nor wraps a perception category that does."); | 197 |
| fail... | 228 |
| fail... | 234 |
| org.opentrafficsim.road.gtu.strategical.LaneBasedStrategicalRoutePlanner | Line |
| this takes the first in the set of links that connects the correct nodes; does not handle parallel links consistently | 204 |
| Probably this test can be removed since we only go "forward" | 209 |
| org.opentrafficsim.road.network.RoadNetwork | Line |
| Is the GTU type actually relevant for physical? It is used still to find adjacent lanes. | 173 |
| isDeadEnd should be removed from LaneChangeInfo, behavior should consider legal vs. physical | 355 |
| org.opentrafficsim.road.network.lane.ConflictTest | Line |
| discuss if this method should be moved into the OtsLine2d class | 212 |
| org.opentrafficsim.road.network.lane.CrossSectionLink | Line |
| work in a different way with lane numbers to align to standard lane numbering. | 57 |
| allow for the removal of a Lane; currently this is not possible. | 69 |
| org.opentrafficsim.road.network.lane.CurveTest | Line |
| finish writing this test | 116 |
| org.opentrafficsim.road.network.lane.DetectorTest | Line |
| this is not true anymore with OperationalPlans, Perception, etc => | 151 |
| the number of events that should be scheduled can vary per models chosen | 152 |
| assertEquals("There should be three scheduled events (trigger, leaveLane, | 153 |
| 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. | 154 |
| assertEquals("Trigger event should be around 38.3403", 38.3403, | 158 |
| triggerEvent.getAbsoluteExecutionTime().get().getSI(), 0.0001); | 159 |
| check that the sensor is triggered at the right time. | 183 |
| org.opentrafficsim.road.network.lane.Lane | Line |
| constructor calls with this(...) | 206 |
| take the cross section slices into account... | 264 |
| take the cross section slices into account... | 280 |
| take the cross section slices into account... | 304 |
| take the cross section slices into account... | 320 |
| 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. | 924 |
| this should return something immutable | 945 |
| determine if this should synchronize on this.nextLanes | 950 |
| And is it aligned with its next lane? | 970 |
| this should return something immutable | 1014 |
| And is it aligned with its next lane? | 1038 |
| let HistoricalArrayList return an Immutable (WRAP) of itself | 1248 |
| org.opentrafficsim.road.network.lane.LaneTest | Line |
| what overtaking conditions do we want to test in this unit test? | 101 |
| This test for expectedLateralCenterOffset fails | 117 |
| This is not correct... | 159 |
| what overtaking conditions do we want to test in this unit test? | 670 |
| org.opentrafficsim.road.network.lane.conflict.ConflictBuilder | Line |
| use z-coordinate for intersections of lines | 51 |
| use remove big parallel type, and use fibers for small tasks. | 52 |
| we cache, but the width generator may be different | 278 |
| handle bus priority on the model side | 502 |
| handle bus priority on the model side | 609 |
| make parallel | 1241 |
| org.opentrafficsim.road.network.lane.object.detector.DirectionalOccupancyDetector | Line |
| enforce clone method | 84 |
| org.opentrafficsim.road.network.lane.object.detector.LoopDetector | Line |
| figure out what possible downstream lanes should get a rear detector | 300 |
| org.opentrafficsim.road.network.lane.object.detector.TrafficLightDetector | Line |
| Possibly provide the GtuTypes that trigger the detector as an argument for the constructor | 105 |
| org.opentrafficsim.road.network.lane.object.sensor.TrafficLightDetectorTest | Line |
| @Test | 150 |
| THIS TEST FAILS!! assertEquals("event list should contain one event (due to creation of the GTU on the detector)", 1, this.loggedEvents.size()); | 235 |
| org.opentrafficsim.road.network.speed.package-info | Line |
| --No comment-- | 21 |
| org.opentrafficsim.road.od.OdApplier | Line |
| sinks? white extension links? | 138 |
| clean up stream acquiring code after task OTS-315 has been completed | 144 |
| should be GTU type dependent. | 636 |
| org.opentrafficsim.road.od.SplitFraction | Line |
| let interpolation interpolate itself | 152 |