Commits on Apr 7, 2024

1. Introduce MoveCooldownHelper to prevent lag spikes from failed pathfi… …

   …nding
   
   Several activities that queue child Move activities can get into a bad scenario where the actor is pathfinding and then gets stuck because the destination is unreachable. When the Move activity then completes, then parent activity sees it has yet to reach the destination and tries to move again. However, the actor is still blocked in the same spot as before and thus the movment finishes immediately. This causes a performance death spiral where the actor attempts to pathfind every tick. The pathfinding attempt can also be very expensive if it must exhaustively check the whole map to determine no route is possible.
   
   In order to prevent blocked actors from running into this scenario, we introduce MoveCooldownHelper. In its default setup it allows the parent activity to bail out if the actor was blocked during a pathfinding attempt. This means the activity will be dropped rather than trying to move endlessly. It also has an option to allow retrying if pathfinding was blocked, but applies a cooldown to avoid the performance penalty. For activities such as Enter, this means the actors will still try and enter their target if it is unreachable, but will only attempt once a second now rather than every tick.

   

   RoosterDragon committed Apr 7, 2024
   Configuration menu

   • View commit details
   • Copy full SHA for 7d4b64c
   • Browse repository at this point

   Copy the full SHA

   7d4b64c View commit details

   Browse the repository at this point in the history