Adding tiebreaking to A* #91

Merged
merged 1 commit into from Apr 4, 2016

Conversation

Projects
None yet
3 participants
@nluqo
Contributor

nluqo commented Apr 1, 2016

Found that small tweaks to A* can dramatically alter the performance in certain cases (linear visits in size of path down from squared). I wrote a few notes with demo here. I tried adding the least obtrusive of the changes (tiebreaking when f scores are equal) and attempted to write a test for it.

This is my first pull request for anything so I apologize if I'm missing something obvious.

@twpage

This comment has been minimized.

Show comment
Hide comment
@twpage

twpage Apr 4, 2016

Interesting! Excellent write up. It's been so long that I actually implemented my own A* - but I'm pretty surprised by these results.

twpage commented Apr 4, 2016

Interesting! Excellent write up. It's been so long that I actually implemented my own A* - but I'm pretty surprised by these results.

@ondras

This comment has been minimized.

Show comment
Hide comment
@ondras

ondras Apr 4, 2016

Owner

Wow, nice and interesting! Thanks for the patch as well as the updated unit test.

Owner

ondras commented Apr 4, 2016

Wow, nice and interesting! Thanks for the patch as well as the updated unit test.

@ondras ondras merged commit e58e99f into ondras:master Apr 4, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment