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.

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

Added test that connects final control from one phase to a parameter of another. #445

Merged
merged 1 commit into from
Nov 12, 2020

Conversation

robfalck
Copy link
Contributor

@robfalck robfalck commented Nov 11, 2020

Summary

Upcoming OpenMDAO 3.4.1 makes it possible to connect the final value of a control from one phase to a parameter value of another. This PR adds a test for this capability.

Related Issues

  • None

Status

  • Ready for merge

Backwards incompatibilities

None

New Dependencies

None

…blre from one phase to a parameter of another. requires OpenMDAO 3.4.1
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 95.805% when pulling d57e4d7 on robfalck:nested_src_indices_updates into c4c7b8a on OpenMDAO:master.

@wright
Copy link
Contributor

wright commented Nov 12, 2020

I don't see any problems with this merge request.

@robfalck robfalck requested a review from wright November 12, 2020 20:31
Copy link
Contributor

@wright wright left a comment

Choose a reason for hiding this comment

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

I don't see any problems with this merge request.

@robfalck robfalck removed the request for review from Kenneth-T-Moore November 12, 2020 20:38
@robfalck robfalck merged commit 703339a into OpenMDAO:master Nov 12, 2020
@robfalck robfalck deleted the nested_src_indices_updates branch December 17, 2020 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants