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 detection of unsatisfiable linkages #502

Merged
merged 4 commits into from
Jan 13, 2021

Conversation

robfalck
Copy link
Contributor

@robfalck robfalck commented Jan 12, 2021

Summary

Adds a check to raise an error when phase linkages are impossible to satisfy due the initial or final state/time being fixed on both sides of a linkage constraint. This does not impact linkages implemented via connection.

Removes some unused non-api functions from load_case.py.

Related Issues

Status

  • Ready for merge

Backwards incompatibilities

None

New Dependencies

None

@coveralls
Copy link

coveralls commented Jan 12, 2021

Coverage Status

Coverage increased (+0.07%) to 96.091% when pulling 66d7655 on robfalck:linkage_compliance into 03c0a6e on OpenMDAO:master.

@robfalck robfalck merged commit 151f1ff into OpenMDAO:master Jan 13, 2021
@robfalck robfalck deleted the linkage_compliance branch November 2, 2021 14:31
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.

Error check for phase linkages to make sure at least one side of it is free to be varied
3 participants