Skip to content

Pathfinding Base #924

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Dec 20, 2018
Merged

Pathfinding Base #924

merged 5 commits into from
Dec 20, 2018

Conversation

creatorfromhell
Copy link
Contributor

  • Allows setting blocked Materials.
  • Allows setting weights to certain Materials.
  • Allows implementation of other algorithms.
  • Initial Algorithm is A*.

@creatorfromhell
Pathfinding Base.
4368015 
- Allows setting blocked Materials.
- Allows setting weights to certain Materials.
- Allows implementation of other algorithms.
- Initial Algorithm is A*.
@CLAassistant
Copy link

CLAassistant commented May 26, 2018 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@mastercoms
Copy link
Member

mastercoms commented May 31, 2018 •
edited
Loading

Thank you for your contribution! I'm all for merging this to just have an implementation for us to experiment with AI behavior and develop it. It can later be replaced with a more robust and performant solution.

Also, maybe refer to #799 for more feedback and ideas, since both PRs seem to use similar algorithms.

@mastercoms mastercoms changed the title Pathfinding Base. Pathfinding Base May 31, 2018
@mastercoms mastercoms mentioned this pull request Jun 8, 2018
Closed
@mastercoms
Copy link
Member

@creatorfromhell What's the progress on this? Do you think it could be reviewed for merge in this state?

@mastercoms mastercoms merged commit 2e5eafd into GlowstoneMC:dev Dec 20, 2018
@mastercoms
Copy link
Member

Thank you for this landmark contribution to Glowstone!

@mastercoms mastercoms mentioned this pull request Jul 15, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Category: gameplay feature Priority: major Status: in progress Type: feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@creatorfromhell @CLAassistant @mastercoms