We were setting a negative velocity tolerance, which never triggered because we
assume tolerances are positive numbers. Now we use a positive tolerance and
assert to catch this case in the future.

Fixes #2765