averbraeck / opentrafficsim

Open Source Multi-Level Traffic Simulator
BSD 3-Clause "New" or "Revised" License
29 stars 11 forks source link

Remove old and unused classes #18

Open averbraeck opened 2 years ago

averbraeck commented 2 years ago

There are many classes in OTS that need to be maintained, but are not in use anymore. Examples are a number of tactical planners (e.g., the LaneBasedGtuFollowingDirectedChangeTacticalPlanner), IdmOld, IdmPlusOld, AbstractLanebasedGtu2, etc. It is suggested to remove these classes

There are also some classes where we have do decide if we want to keep them in the current version, or remove them for now and add them back later if needed. Examples are Platoons, the Toledo classes, and several other unused (and probably unmaintained) classes.

averbraeck commented 2 years ago

Already removed the following classes in the branch feature-remove-old:

averbraeck commented 2 years ago

Doubts about the following classes, since they are in use in several places:

In general, I am all in favor of implementing 'textbook' algorithms into OTS for teaching purposes (like instantaneous lane change). But they should be built on exactly the same perception framework as the more advanced algorithms, but just use a smaller percentage of the information that is available to make their decisions, resulting is more simplistic and less optimal driving behavior.

@WJSchakel : any ideas what to keep, what to upgrade, what to delete?

averbraeck commented 2 years ago

The branch feature-remove-old has been merged with the main branch. So a number of the classes 'without side-effects' have been removed.