Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swapped indexes in astar.h #24

Closed
fwanderlingh opened this issue May 16, 2014 · 2 comments
Closed

Swapped indexes in astar.h #24

fwanderlingh opened this issue May 16, 2014 · 2 comments

Comments

@fwanderlingh
Copy link

Hey man, while using your fantastic rep I encountered a bug in the astar header, in particular the indexing of the grid in the neighbor search when using nx,ny and cx,cy where swapped. That was causing the algorithm to search blocks that could be occupied by walls if the map was different from the sample one. I'm using your astar header in one of my repositories, so if you want feel free to have a look at the changes I've done 😉

I'm not using the github tools to fork the rep and request a merge since I'm somehow new to git and I don't want to create a mess 👅

@xtaci
Copy link
Owner

xtaci commented May 17, 2014

yes ,it's true.
i'll try to fix this problem. if you get familiar with git , you can just simply "pull requests".
thanks guy!

@xtaci
Copy link
Owner

xtaci commented May 17, 2014

I've done some optimization to the astar algorithm, and also some comments.

@xtaci xtaci closed this as completed May 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants