OpenMDAO / dymos

Open Source Optimization of Dynamic Multidisciplinary Systems
Apache License 2.0
202 stars 65 forks source link

Removed redundant implementations of timeseries output components. #1005

Closed robfalck closed 10 months ago

robfalck commented 10 months ago

Summary

All transcriptions now share the same TimeseriesOutputComp from transcriptions/common. Added a comparison dunder method eq to GridData so that we can quickly test if two grid data objects have the same data even if they are separate instantiations. Some minor cleanup was necessary in SolveIVPTimeseriesOutputComp even though all SolveIVP machinery is deprecated through not yet removed until dymos 2.0.

Related Issues

Backwards incompatibilities

None

New Dependencies

None

coveralls commented 10 months ago

Coverage Status

coverage: 92.686% (+0.1%) from 92.55% when pulling f85969ecce70522f071407665a0a4d85661e7dee on robfalck:timeseries_dedupe into 2953f09c5bd5bfac5285a0a41f2857f6dcdcb23a on OpenMDAO:master.