Mirrored_Repos/veloren
1
0
Fork 0
mirror of https://gitlab.com/veloren/veloren.git synced 2024-08-30 18:12:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix heuristic to restart route in chase()

Browse Source 
We had a random chance of restarting the route to avoid getting the
chaser stuck. This didn't work properly because the rerouting code was
only triggered if `self.route.is_none()` or if the target moved
significantly, and the random chance branch only set `bearing` to `None`.

This change sets `self.route = None` to trigger rerouting.
This commit is contained in:
Hugo Peixoto 2022-05-14 14:50:09 +01:00
parent ba6d1bce60
commit b681ad79ce
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
common/src/path.rs
View File

@ -380,9 +380,10 @@ impl Chaser {
// theory this shouldn't happen, but in practice the world is full // theory this shouldn't happen, but in practice the world is full
// of unpredictable obstacles that are more than willing to mess up // of unpredictable obstacles that are more than willing to mess up
// our day. TODO: Come up with a better heuristic for this // our day. TODO: Come up with a better heuristic for this
if (end_to_tgt > pos_to_tgt * 0.3 + 5.0 && complete) if end_to_tgt > pos_to_tgt * 0.3 + 5.0 && complete {
|| thread_rng().gen::<f32>() < 0.001 None
{ } else if thread_rng().gen::<f32>() < 0.001 {
self.route = None;
None None
} else { } else {
self.route self.route