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 | 91 | TODO, FIXME, XXX |
Each tag is detailed below:
Open issues in the code
Number of occurrences found in the code: 91
| 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()); | 140 |
| removeLane should throw an Error when the car is not on that lane (currently this is silently ignored) | 295 |
| figure out why the added lane has a non-zero position | 296 |
| org.opentrafficsim.road.gtu.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 | 245 |
| instead of getNetwork().getNodeMap().values(), using spatial tree would be a good alternative perhaps even a findClosest() method. | 532 |
| org.opentrafficsim.road.gtu.LaneBasedGtuTest | Line |
| see how we can ask the vehicle to look this far ahead | 182 |
| see how we can ask the vehicle to look this far behind | 189 |
| see how we can ask the vehicle to look 'forwardMaxDistance' ahead | 222 |
| follower = truck.headway(reverseMaxDistance); | 243 |
| org.opentrafficsim.road.gtu.control.DelayedActuation | Line |
| numerical implementation of tau rule | 43 |
| org.opentrafficsim.road.gtu.generator.Platoons | Line |
| end time of platoon may be in next demand period, which makes the demand non-linear | 321 |
| 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.perception.mental.channel.ChannelTaskIntersection | Line |
| add @see ref to paper by Yiyun et al. | 76 |
| org.opentrafficsim.road.gtu.perception.mental.channel.IntersectionPerceptionChannel | Line |
| visibility | 137 |
| org.opentrafficsim.road.gtu.perception.object.PerceivedConflict | Line |
| stop lines (current models happen not to use this, but should be possible) | 161 |
| 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.gtu.tactical.lmrs.AccelerationBusStop | Line |
| this process is much more complex: tail blocking other traffic? other bus in front? many people at bus stop? | 36 |
| org.opentrafficsim.road.gtu.tactical.lmrs.AccelerationNoSlowLaneOvertake | Line |
| depends on left/right traffic | 62 |
| depends on left/right traffic | 79 |
| org.opentrafficsim.road.gtu.tactical.lmrs.IncentiveCourtesy | Line |
| factor -a/b as influence factor is heavy in calculation, consider v<vEgo & 1-s/x0 | 88 |
| org.opentrafficsim.road.gtu.tactical.lmrs.IncentiveQueue | Line |
| a ramp-metering traffic light triggers this incentive with possible cooperation from the main line possible solution: make this a state-full class using Lane/LinkTypes to recognize intersections | 72 |
| org.opentrafficsim.road.gtu.tactical.lmrs.IncentiveSocioSpeed | Line |
| keep left or right rules | 38 |
| org.opentrafficsim.road.gtu.tactical.lmrs.IncentiveSpeed | Line |
| SpeedWithCourtesy now uses TrafficPerception, which embeds the courtesy part. How to do this? | 49 |
| org.opentrafficsim.road.gtu.tactical.lmrs.Lmrs | Line |
| consider route in incentives (only if not on current lane?) | 107 |
| org.opentrafficsim.road.gtu.tactical.util.ConflictUtil | Line |
| do not ignore vehicles upstream of conflict if they have green | 78 |
| conflict over multiple lanes (longitudinal in own direction) | 79 |
| 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) | 80 |
| priority rules of busses should be handled differently This also makes a GTU type and -Ego- context unnecessary here | 243 |
| depending on left/right-hand traffic | 254 |
| safety factor? | 656 |
| this check is simplistic, designed quick and dirty, it just adds 3s as a safe gap | 713 |
| stopping | 911 |
| all-stop behavior | 923 |
| org.opentrafficsim.road.gtu.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 | 96 |
| org.opentrafficsim.road.gtu.tactical.util.lmrs.GapAcceptance | Line |
| --No comment-- | 52 |
| org.opentrafficsim.road.gtu.tactical.util.lmrs.LmrsUtil | Line |
| this approach is a hack | 426 |
| org.opentrafficsim.road.gtu.tactical.util.lmrs.Synchronization | Line |
| remove infrastructure constraints (that is up to DeadEndUtil) | 186 |
| select first leader towards which the acceleration is > -b0 (rather than b) | 187 |
| org.opentrafficsim.road.gtu.tactical.util.lmrs.Tailgating | Line |
| direction depends on left/right traffic | 180 |
| org.opentrafficsim.road.network.CrossSectionLink | Line |
| work in a different way with lane numbers to align to standard lane numbering. | 56 |
| allow for the removal of a Lane; currently this is not possible. | 68 |
| org.opentrafficsim.road.network.Lane | Line |
| constructor calls with this(...) | 202 |
| take the cross section slices into account... | 260 |
| take the cross section slices into account... | 276 |
| take the cross section slices into account... | 300 |
| take the cross section slices into account... | 316 |
| 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. | 916 |
| this should return something immutable | 937 |
| determine if this should synchronize on this.nextLanes | 942 |
| And is it aligned with its next lane? | 962 |
| this should return something immutable | 1006 |
| And is it aligned with its next lane? | 1030 |
| let HistoricalArrayList return an Immutable (WRAP) of itself | 1189 |
| org.opentrafficsim.road.network.RoadNetwork | Line |
| Is the GTU type actually relevant for physical? It is used still to find adjacent lanes. | 171 |
| isDeadEnd should be removed from LaneChangeInfo, behavior should consider legal vs. physical | 353 |
| org.opentrafficsim.road.network.StripeData | Line |
| this is a bad way to define types and mutable data | 70 |
| org.opentrafficsim.road.network.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 | 503 |
| handle bus priority on the model side | 610 |
| make parallel | 1242 |
| org.opentrafficsim.road.network.lane.ConflictTest | Line |
| discuss if this method should be moved into the OtsLine2d class | 217 |
| org.opentrafficsim.road.network.lane.CurveTest | Line |
| finish writing this test | 118 |
| org.opentrafficsim.road.network.lane.DetectorTest | Line |
| this is not true anymore with OperationalPlans, Perception, etc => | 157 |
| the number of events that should be scheduled can vary per models chosen | 158 |
| assertEquals("There should be three scheduled events (trigger, leaveLane, | 159 |
| 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. | 160 |
| assertEquals("Trigger event should be around 38.3403", 38.3403, | 164 |
| triggerEvent.getAbsoluteExecutionTime().get().getSI(), 0.0001); | 165 |
| check that the sensor is triggered at the right time. | 189 |
| org.opentrafficsim.road.network.lane.LaneTest | Line |
| what overtaking conditions do we want to test in this unit test? | 106 |
| This test for expectedLateralCenterOffset fails | 122 |
| This is not correct... | 164 |
| what overtaking conditions do we want to test in this unit test? | 675 |
| org.opentrafficsim.road.network.lane.object.sensor.TrafficLightDetectorTest | Line |
| @Test | 152 |
| THIS TEST FAILS!! assertEquals("event list should contain one event (due to creation of the GTU on the detector)", 1, this.loggedEvents.size()); | 237 |
| org.opentrafficsim.road.network.object.detector.DirectionalOccupancyDetector | Line |
| enforce clone method | 84 |
| org.opentrafficsim.road.network.object.detector.LoopDetector | Line |
| figure out what possible downstream lanes should get a rear detector | 302 |
| org.opentrafficsim.road.network.object.detector.TrafficLightDetector | Line |
| Possibly provide the GtuTypes that trigger the detector as an argument for the constructor | 105 |
| 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 |