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

MPhys wrapper: Standardize and cleanup the pygeo wrapper #122

Open
anilyil opened this issue Mar 13, 2022 · 0 comments
Open

MPhys wrapper: Standardize and cleanup the pygeo wrapper #122

anilyil opened this issue Mar 13, 2022 · 0 comments
Labels

Comments

@anilyil
Copy link
Contributor

anilyil commented Mar 13, 2022

Description of feature

the pygeo wrapper can be cleaned up substantially and we can also make the integration with the new api smoother.

In particular, during reverse mode derivative computations, it does unnecessary work when we have many thickness constraints, but pygeo still evaluates the full jacobian including the cfd surface nodes even though their seeds are zero.

Moved the issue from the mphys repo: OpenMDAO/mphys#62

Potential solution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants