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

add extra cost for turns in A* to avoid staircase #32

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

reimai
Copy link

@reimai reimai commented May 25, 2013

No description provided.

@guilhermesimoes
Copy link

Can't comment on the pull request itself, but you are using tabs when the project is using spaces for indentation. You should always follow the project's indentation style.

@qiao
Copy link
Owner

qiao commented May 26, 2013

Hi @reimai, thanks for your pull request. A minor issue with your patch is that you should modify the source file located in the src directory instead of the compiled file. Then use the make command to rebuild the browser version.(You may need to install nodejs and some dependent libraries).

@ivanmayes
Copy link

ivanmayes commented Jul 25, 2017

Integrated this into src at #150 as well as adding some configuration for the turn penalty number.

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

Successfully merging this pull request may close these issues.

5 participants