OpenMDAO / dymos

Open Source Optimization of Dynamic Multidisciplinary Systems
Apache License 2.0
210 stars 67 forks source link

Added optional `integ_var_options` for `time_options` when the integration variable is not time. #1079

Closed robfalck closed 4 months ago

robfalck commented 4 months ago

Summary

Added phase.integ_var_options, phase.set_integ_var_options, and phase.set_integ_var_val as aliases for phase.time_options, phase.set_time_options, and phase.set_time_val, respectively. These aliases might seem more natural when the integration variable of the phase is not time.

Related Issues

Backwards incompatibilities

None

New Dependencies

None

review-notebook-app[bot] commented 4 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

coveralls commented 4 months ago

Coverage Status

coverage: 92.57% (-0.2%) from 92.779% when pulling 60802b607beca80fbfd5a62c705538325d330aa9 on robfalck:integ_var_options into ddb8627ab8cdfd0f94dc2ca5d39b02a9b304bb19 on OpenMDAO:master.