OpenMDAO / dymos

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

Default value for `times_per_seg` should be a simulation option for phases. #992

Closed robfalck closed 11 months ago

robfalck commented 11 months ago

Proposed feature.

Currently times_per_seg is specified as an argument to simulate. If not provided, the default value should be in SimulateOptionsDictionary/Phase.simulation_options.

For some transcription types, more times per segment would be preferable by default.

Example

Currently the user cannot do something like

phase.set_simulation_options(times_per_seg=50)