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

rewrite of FindPath #554

Merged
merged 4 commits into from Oct 23, 2023
Merged

rewrite of FindPath #554

merged 4 commits into from Oct 23, 2023

Conversation

turleypol
Copy link
Member

Uses now position/range classes
Extended the params to not only have the blocking spots but all common things per node to minimize the memory per node.
General modernization
added parameter for movemode ("LSA") before it was only possible to use findpath on land
Testscript added.

added parameter for movemode, before it was only possible to use
findpath on land
@coveralls
Copy link
Collaborator

coveralls commented Oct 19, 2023

Coverage Status

coverage: 46.092% (+0.5%) from 45.543% when pulling 2f5a5a6 on pathfind_vector into 31780b0 on master.

@turleypol turleypol marked this pull request as ready for review October 22, 2023 14:30
Copy link
Contributor

@KevinEady KevinEady left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🎆

@turleypol turleypol merged commit 4895bfd into master Oct 23, 2023
68 checks passed
@turleypol turleypol deleted the pathfind_vector branch October 23, 2023 17:35
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.

None yet

3 participants