RoosterDragon ac55c5bf09 Fix pathfinding using PriorityQueue incorrectly. ...

By providing a comparer that could change over time (as estimated costs on the graph were updated), this meant the priority queue could have its heap property invalidated and thus not maintain a correct ordering. Instead we store elements into the queue with their estimations at the time. This preserves the heap property and thus ensures the queue returns properly ordered results, although it may contain out of date estimations.

This also improves performance. The fixed comparer need not perform expensive lookups into the graph, but can instead use the readily available value. This speeds up adds and removes on the queue significantly.

2015-09-01 17:29:36 +01:00

1.4 KiB

Raw Blame History

View Raw