Skip to content

Multiple variable input for add_timeseries_output #387

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
Merged
merged 6 commits into from
Sep 16, 2020
Merged

Multiple variable input for add_timeseries_output #387

merged 6 commits into from
Sep 16, 2020

Conversation

wright
Copy link
Contributor

@wright wright commented Sep 15, 2020

Summary

Updates for add_timeseries_output() to allow:

  • Wildcard inputs
  • List inputs
  • Dictionary or List units when using List inputs

Related Issues

Status

  • Ready for merge

Backwards incompatibilities

None

New Dependencies

None

@coveralls
Copy link

coveralls commented Sep 15, 2020 •
edited
Loading

Coverage Status

Coverage increased (+0.03%) to 95.472% when pulling 01bfdb4 on wright:master into c168731 on OpenMDAO:master.

@wright wright requested a review from robfalck September 15, 2020 15:21
@wright
Copy link
Contributor Author

wright commented Sep 16, 2020

There seems to be an issue with wildcards that contain a period, such as:
phase.add_timeseries_output('aero.*')
I am looking into it.

@robfalck robfalck merged commit f79faaa into OpenMDAO:master Sep 16, 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

@Kenneth-T-Moore Kenneth-T-Moore Kenneth-T-Moore 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.

add_timeseries_output should allow multiple variables to be added per call
4 participants
@wright @coveralls @robfalck @Kenneth-T-Moore