-
Notifications
You must be signed in to change notification settings - Fork 47
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
Check the interpolators and use more efficient versions where applicable. #37
Conversation
Should I just switch all of our workflows over to use the OpenMDAO dev branch, once my fix there is merged? It will be a while before the next release. |
I think this is worth discussing in the Aviary dev meeting. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Putting a hold on this until the pypi release
@@ -55,6 +55,16 @@ jobs: | |||
auto-update-conda: true | |||
python-version: ${{ matrix.PY }} | |||
|
|||
- name: Install OpenMDAO Dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
filename: test_workflow_no_dev_install.yml
test name: Install OpenMDAO Dev
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, with this PR, we can't run anything without dev. My choices would be to remove this workflow, or just have it load "dev" until the next release, and then remove that task to go back to a no-dev run. I went with the latter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was just pointing out the irony, not requesting changes
.github/workflows/test_workflow.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with the pypi release, can these be changed back from dev?
Summary
swap
benchmarks that was failing.Related Issues
Backwards incompatibilities
This PR requires the following PR in OpenMDAO, which fixes a bug in the 2D-slinear interpolation:
OpenMDAO/OpenMDAO#3094
New Dependencies
None