averbraeck / opentrafficsim

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

Clean up ConflictBuilder and ConflictBuilderParallel #80

Closed WJSchakel closed 10 months ago

WJSchakel commented 1 year ago

The classes ConflictBuilder and ConflictBuilderParallel are huge classes with considerable overlap, between them and within each. The parallel code that was added is not well documented. It should be investigated whether the code is ready. There are big tasks (from one lane, consider all later in list of lanes), and small tasks (consider specific lane combo), what to do with these? The builders are also quite verbose.

WJSchakel commented 10 months ago

ConflictBuilderParallel was removed. Documentation in ConflictBuilder was improved, particularly on the parallel implementations. There are two parallel implementations:

This issue will be closed now. In the future it may be re-opened to: