Trajectory results report plots now plot vector variables, respect the x_name argument, and avoid noisy plots when values are nearly constant. #1135
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
The bokeh traj results report created by run_problem was not respecting plot_kwargs. This is now fixed, allowing users to specify the name of the time-like variable when phases rename the time-like variable.
Vectorized states, controls, and parameters were previously not being plotted correctly in the traj results report.
For example, in the brachistochrone vector states test,
pos
is a 2-vector holding both x and y states.Previously, these weren't plotted correctly because Bokeh's ColumnDataSource seems to prefer a single column of data for each variable name.
Now the report will include a new figure for each element in the vector, with the axis labeled correctly.
Finally, in some cases, machine precision variation in otherwise-constant data would show up as noisy.
The y-range of the plots has now had its
min_interval
set to1.0E-12
so that. machine-precision variation, typically on the order of 1E-16, appears constant.Related Issues
Backwards incompatibilities
None
New Dependencies
None