-
Notifications
You must be signed in to change notification settings - Fork 3
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
Generate true path heuristic for STA* (save this for each scenario) #6
Comments
Testing existing euclidean heuristic vs true heuristic on a fresh world 32 x 36 grid, 64 robots, 48 item zones, 28 stations. 500 new orders all at once. Euclidean Gets stuck after around 700 steps with 17 finished orders, with robot allocator failing to find paths (500 max time, 10,000 max iters) for robots. Generally generate_path was taking > 100ms so only 1 robot job was processed per time step True Heuristic (takes ~3.5 seconds to pre-compute) Looks to be functioning well, most robots being activated and used
Somewhat of a bi-modal distribution for both, but we clearly see the true heuristic takes around 1/10th the time of the euclidean heuristic (and also completes where the other fails on maxiter(?)) Bi-modal is possibly due to astar failing out quickly due to a dynamic obstacle blocking all cells or similar |
With this change, I expected to see CPU relatively unchanged (instead of a quick math calc, it's doing a hash lookup + 2d array lookup per heuristic call), and slightly increased memory for the size of the heuristic table. What we see is prior had lower CPU usuage, and higher memory usage, so opposite. This sample may be unrelated perhaps. docker ps on GCE micro instance before change
After
Looking into the change we also increased max time steps from 200 to 500, so while previously the euclidean was failing out at finding a path and taking >100ms for a single search, the new true heuristic is likely doing much more computation and finding actual paths, with enough time for multiple robots. After things have settled looks like it's no longer an issue
|
No description provided.
The text was updated successfully, but these errors were encountered: