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

Refactor astar pt3 #3650

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

Conversation

dkargin
Copy link
Contributor

@dkargin dkargin commented Feb 23, 2024

No description provided.

Current A* implementation is tailored for single unit/single path
requests. But can provide a wider functionality, like calculation of
shared flow field, or fast distance transform or group path finding.

Current changes are trying to make this component more generic:
 - A* wave propagation now uses separate predicate function, which
provides checks for goal and heuristic costs
 - Path tracing moved to a standalone function
1. Wrapped PF map cache into a standalone structure/instance
2. Moved all global state out of astar.cpp.
3. Using backward path search. It provides more opportunities for reusing
previous contexts.
4. Introduced cost accessor to A* wave exploration
Implemented debug rendering for pathginding:
 - renderer for blocking layer
 - rendering of current paths
 - drawing of path context
@ManManson ManManson self-requested a review March 13, 2024 17:43
@KJeff01 KJeff01 mentioned this pull request Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants