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

question about distable #11

Closed
ShuaiZhou302 opened this issue Jun 11, 2024 · 2 comments
Closed

question about distable #11

ShuaiZhou302 opened this issue Jun 11, 2024 · 2 comments

Comments

@ShuaiZhou302
Copy link

Hello sir
sorry to bother you again
In your pypibt, you used bfs to generate distable
First question is:
I wonder if you still using it in the C++ pibt2
when i check solver.cpp, you are using bfsto generate distable too.
but when i run my reproduction code(which is confirmed no problem) in python, it is very slow in generating distable part when i use ost003d.map this kinds of big size map in (mapf benchmark){https://movingai.com/benchmarks/mapf/index.html} it's fast in searching process still, the other parts is ok. about 40ms averagely

and my second question is based on first.
if you are not using bfs, what did you use.
if you still using bfs , is the slow generating process just the difference between python and C++, or you have the same problem, but it doeson't counted on runtime for it is not part of searching process.

Thank you so much if you could respond to me!
and thanks again! for your earlier kind answer.

@Kei18
Copy link
Owner

Kei18 commented Jun 11, 2024

Hi,

Calculating the distance table is a necessary overhead; even with C++, this process can take time compared to running PIBT itself. So Python may suffer more from this.

An alternative is to use A* (or other pathfinding methods) to compute the distance-to-go on demand, according to the query of the vertex evaluation in PIBT. But I do not recommend this strategy for speed reasons. You can also check RRA* (same paper as HCA*).

Overall, simple BFS is the fastest way in my experience...

Best,
Keisuke

@ShuaiZhou302
Copy link
Author

Thank you!

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

No branches or pull requests

2 participants