resonance icon indicating copy to clipboard operation
resonance copied to clipboard

[WIP] Use new solve_ivp for ODE integration.

Open ixjlyons opened this issue 6 years ago • 1 comments

Initial working implementation in place. Some notes:

  • scipy 1.0 isn't available via conda-forge yet
  • All the integrators seem to be either super slow or unstable for the CoulombPendulumSystem if you integrate out beyond the sticking point for much time. The fixed time step Runge-Kutta solver we're using currently is nice there though it has accuracy issues. Could just bring that back I suppose.
  • Currently the integration method is specified via an attribute on the system class. It should probably at least be a property with a setter that raises an ValueError if you try to set it to something not available.
  • I think the ode_func is intended to always be vectorized, but I'm guessing some fun traceback will result if it's not. Should probably handle that by passing in dummy values when it's set.

Fixes #102

ixjlyons avatar Oct 31 '17 01:10 ixjlyons