Avoid unnecessary turns in path finder #196
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This introduces a new type HexCoord::Difference, which represents not absolute coordinates but differences between them (i.e. directions or steps). While both of them are quite similar, they are used differently in e.g. the arithmetic operations. Previously both were using the same type, but this change enforces using the right type in each place now.
When stepping paths, try to continue the last direction as long as possible while still retaining the optimal distance. This helps to avoid unnecessary turns (which then lead to unnecessary many waypoints in moves). Note that the implementation is very simple and just a greedy approach, so it is not (?) guaranteed to really minimise the number of turns. It does so quite well in practice, though. Since the path finder is not part of consensus, this change is not forking the game state (nor would further tweaks to the algorithm cause forks).
domob1812
force-pushed
the
master
branch
4 times, most recently
from
0852243 to
45886a9
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This updates the logic in the path finder for stepping the path once the distance map has been computed. Instead of always choosing the next direction based on a fixed order of all six principal directions, we try to stick to the previous direction as long as possible. This is a greedy approach to minimise the number of turns taken, and thus the number of waypoints and sizes of moves. The resulting paths are still of optimal length.
This does not affect consensus (nor would future tuning of the algorithm), since path finding is no longer part of consensus since #145).