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

Move dvcon to a matrix-based API for partials #69

Closed
anilyil opened this issue Jun 29, 2021 · 2 comments
Closed

Move dvcon to a matrix-based API for partials #69

anilyil opened this issue Jun 29, 2021 · 2 comments
Assignees

Comments

@anilyil
Copy link
Collaborator

anilyil commented Jun 29, 2021

With DVConstraints, and possibly with DVGeometry wrapper as well, we compute and store a full Jacobian internally using complex step or parallel finite differences. However, we have been using the matrix-free Jacobian vector product API in OpenMDAO instead of the matrix-based API.

Our approach is not well suited with the mat-free API; we end up either needing to do full jacobian computations each time mat-free API is called, or we need to keep track of design changes and update the Jacobians once and re-use them for subsequent mat-free API calls.

Matrix-based API is the way to go here: OpenMDAO would automatically update the matrix-based Jacobians when the design is changed. The only work we need to do here is to expose our Jacobians to OpenMDAO in the correct format for each DV and output/constraint.

@anilyil
Copy link
Collaborator Author

anilyil commented Jun 29, 2021

See a brief discussion of this here: #68 (comment)

@anilyil
Copy link
Collaborator Author

anilyil commented Mar 13, 2022

Moved to the pyGeo repo: mdolab/pygeo#123

@anilyil anilyil closed this as completed Mar 13, 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

No branches or pull requests

1 participant