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

Update test_adjoint_compare.py #60

Merged
merged 2 commits into from
Jun 23, 2020
Merged

Update test_adjoint_compare.py #60

merged 2 commits into from
Jun 23, 2020

Conversation

DavidAnderegg
Copy link
Collaborator

Purpose

Since the package name was updated, this should be updated accordingly.

Type of change

What types of change is it?

  • Bugfix (non-breaking change which fixes an issue)

Testing

Run the file "run_om_reg_tests.py".

Checklist

Put an x in the boxes that apply.

  • I have run unit and regression tests which pass locally with my changes
  • I have added new tests that prove my fix is effective or that my feature works
  • I have added necessary documentation

Since the package name was updated, this should be updated accordingly.
@DavidAnderegg DavidAnderegg requested a review from a team as a code owner June 23, 2020 07:04
@ewu63 ewu63 merged commit 3bc7de2 into mdolab:master Jun 23, 2020
@DavidAnderegg DavidAnderegg deleted the patch-1 branch June 23, 2020 14:26
@anilyil
Copy link
Contributor

anilyil commented Jun 23, 2020

@DavidAnderegg, we do not actively maintain the OpenMDAO wrapper that is in this repo. We are working on another version with a few improvements. Feel free to keep using this, but we suggest you perform your own testing to make sure this wrapper works. We will release the new wrapper once it is more or less finalized and ready for general use.

@DavidAnderegg
Copy link
Collaborator Author

@anilyil, Thank you for your reply.

Do you have any idea when that release will happen? I am eager to test it out.

@anilyil
Copy link
Contributor

anilyil commented Jun 24, 2020

Thanks for the interest @DavidAnderegg. We will release it possibly some time over the summer. The updated ADflow wrapper inherited a lot of the code from the current wrapper. Hopefully the transition to the new version will be smooth. Keep an eye out for issue #50. We will update that with the progress related to this.

marcomangano pushed a commit to marcomangano/adflow that referenced this pull request Aug 19, 2020
Co-authored-by: Neil Wu <neilwu@umich.edu>
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