Popmotion: bounce animation with some durations / velocity updates to NaN

[jedierikb]

When starting a bounce animation with some configurations of duration and velocity can result in NaN update values.
(The animations do successfully end at their to values though.)

For example:
https://codesandbox.io/s/popmotion-velocity-nan-et3t3?file=/src/index.tsx:0-425

import { animate } from "popmotion";

const $root = document.getElementById('root');

const from = 1;
const to = 5;
const duration = 4000;
const type = 'spring';
const bounce = 0.5;
const velocity = 0.9988001159034534;

animate({
  from: from,
  to: to,
  duration: duration,
  type: type,
  bounce: bounce,
  velocity: velocity,
  onUpdate: latest => $root.innerHTML = 
  `from: ${from}<br>to ${to}<br>latest: ${latest}`
})

This animation should not have NaN values onUpdate

It seems that the actual Nan is created hereabouts:

popmotion/packages/popmotion/src/animations/generators/spring.ts

Line 75 in a1903c8

function createSpring() {

as a result of an invalid dampingRatio.

Please let me know how I could help with this issue. Thank you.
This issue was initially discovered here
motiondivision/motion#1094

[jedierikb]

Seems to happen when dampingRatio becomes NaN as a result of 0/0 here:

popmotion/packages/popmotion/src/animations/generators/spring.ts

Line 78 in a1903c8

const dampingRatio = damping / (2 * Math.sqrt(stiffness * mass))

Where the damping is 0 and the stiffness is 0.

And it seems 0 values can come from approximateRoot

popmotion/packages/popmotion/src/animations/utils/find-spring.ts

Line 98 in a1903c8

function approximateRoot(

[jedierikb]

Can I help clarify this issue or help to patch?