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

Parameter flow added to linkage report diagram #873

Merged
merged 20 commits into from
Nov 23, 2022
Merged

Conversation

tadkollar
Copy link
Member

Summary

There are new nodes labeled "params" in the diagram which contains the parameters for the trajectory and each phase. The trajectory parameters node is at the same level as the phases, and the phase parameters appear at the same level as the initial/final conditions.

The only intentional difference between this implementation and the issue description is that the parameters start out minimized due to the large number of them in some models. If one of a parent's child nodes was yellow, the collapsed parent will be yellow; however if a different child was red, the parent will be red.

Here is the report with the trajectory parameters and those in rotate manually expanded:
image

Related Issues

Backwards incompatibilities

None

New Dependencies

None

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar
…evel color of its children

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar

Verified

This commit was signed with the committer’s verified signature.
tadkollar Tad Kollar
@robfalck robfalck merged commit 7264796 into OpenMDAO:master Nov 23, 2022
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.

Add parameter flow to the linkage report diagram.
2 participants