Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

I402 Automatically generate plots of all timeseries outputs #469

Merged
merged 18 commits into from
Dec 2, 2020
Merged

I402 Automatically generate plots of all timeseries outputs #469

merged 18 commits into from
Dec 2, 2020

Conversation

hschilling
Copy link
Member

Summary

Provides a way to easily create simple plots of the implicit solution and explicit simulation results for all timeseries outputs.
The plots include multiple phases on a single set of axes by default.
The different phases are plotted uniquely using colors from a colormap of 20 colors.
Plots are placed in a user-settable subdirectory.
Plotting is done from recorded files, not live problem objects.
The user can generate plots in one of three ways:

  1. from the command line interface
  2. using the run_problem function
  3. using the timeseries_plots function

Related Issues

Status

  • Ready for merge

Backwards incompatibilities

None

New Dependencies

None

@hschilling
Initial code to add timeseries plots
257f0c5
@hschilling
fixed merge conflicts
42a7f8a
@hschilling
remove visualization from the ignores
66e3db3
@hschilling
visualization folder
abf40a1
@hschilling
removed files that should not have been committed
8c8fc0a
@hschilling
Fixed location and spacing for legend
183f9e5
@hschilling
All code and tests for basic plotting from run_problem and timeseries…
9ecbd60 
…_plots work
@hschilling
Many changes to get to the point of being able to plot multiple phases
bcb8af4
@hschilling
All three example codes work now when plotting using the function
b0f4aa0
@hschilling
change in comparison to eliminate a warning message
5727ad9
@hschilling
Large amount of code cleanup and PEP8 fixes
23f36af
@hschilling
Merge branch 'master' of https://github.com/OpenMDAO/dymos into I402-…
98b1e44 
…timeseries-plots
@hschilling
Added doc for timeseries_plot function
f0e81d9
@hschilling
Final cleanup before PR
213cd55
@hschilling
Removed commented out code
4280f6d
@coveralls
Copy link

coveralls commented Nov 25, 2020 •
edited
Loading

Coverage Status

Coverage increased (+0.06%) to 96.186% when pulling 40fc426 on hschilling:I402-timeseries-plots into e14fe44 on OpenMDAO:master.

@robfalck robfalck self-requested a review November 30, 2020 12:47
robfalck
robfalck requested changes Nov 30, 2020
View reviewed changes
Copy link
Contributor

@robfalck robfalck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm generally really happy with this. Aside from the filename nit pick, it should plot every variable in the timeseries, not just the states and time.

@robfalck robfalck self-requested a review December 2, 2020 19:33
@robfalck robfalck merged commit 53cc645 into OpenMDAO:master Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robfalck robfalck robfalck approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Automatically generate plots of all timeseries outputs
3 participants
@hschilling @coveralls @robfalck