Added interpolation of results to arbitrary nodes, removed use of `scipy.interpolate.interp1d` from `phase.interp`.

robfalck commented 2 weeks ago

Summary

Per a users request, added the ability to use phase.interp to obtain outputs on any set of points in phase tau space [-1, 1].
Switched phase.interp to use scipy.interpolate.make_interp_spline. Spline types that are no longer supported will now result in linear interpolation.
remove deprecated Phase.interpolate method. users should use Phase.interp.
removed outdated test_upgrade_guide tests.
changed cannonball implicit duration test to use ScipyOptimizeDriver.

In general IPOPT on the latest workflow seems to be having issues with converging some of the problems, but it's not obvious yet as to why. Need to keep an eye on it.

Related Issues

Resolves #

Backwards incompatibilities

-Phase.interpolate is removed in favor of Phase.interp.

New Dependencies

None

review-notebook-app[bot] commented 2 weeks ago

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

coveralls commented 1 week ago

coverage: 93.667% (-0.3%) from 93.922% when pulling c61f09a0acca37d1c489a1c4f3231cbe506cef30 on robfalck:interp_update into bb341a43d392dca226bd506af372bc83bf88e5f5 on OpenMDAO:master.

OpenMDAO / dymos