Bug report

Required Info:

• Version or commit hash:
  • rolling head (13f728a)

Expected behavior

For all BT nodes, tick should return quickly.

Actual behavior

IsPathValid has spin_until_future_complete, which can take several ms to complete (that's a lot for a BT to be ticked at 100 Hz).

Because this calls an action, its behavior should be equivalent to the action nodes inheriting from BtActionNode, returning RUNNING while the action is calculating. I've checked, I didn't see such spinning in a tick function in other condition nodes. Should this be transformed into an action node to prevent code duplication?