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

AI pathfinding cache #411

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

Conversation

Doubleumc
Copy link
Contributor

About the pull request

Pathfinding keeps a cache of goal searches planned that tick.
Each individual goal still needs pathed to but each agent can reuse parts of a path to that target generated by another agent.

Explain why it's good for the game

Should reduce duplicated work per-AI on pathing to targets and hopefully reduce AI's performance overhead.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
code: pathfinding caches goal searches for better performance
/:cl:

@Doubleumc Doubleumc added do not merge needs tested testmerge this to see what breaks labels Aug 25, 2024
@Zonespace27
Copy link
Contributor

Mirroring from discord so it doesn't get lost:
Had pathfinding cache TMed for an operation of mine, and had the following results:

  • Xenos sometimes became incredibly slow, to the degree of being easily outpaced by marines in heavy armor
  • Xenos bumped into walls and got lost a ton more, but this might just be their AI in general

@Doubleumc Doubleumc marked this pull request as draft August 29, 2024 01:06
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement do not merge needs tested testmerge this to see what breaks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants