-
Notifications
You must be signed in to change notification settings - Fork 9
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 hiop interface due to the latest changes in hiop #30
Conversation
cbf45df
to
bfda398
Compare
I think we want #20 merged before this, and perhaps need to resolve #10 for Ascent in CI to make this trivial to re-build and test. Thanks for the fix @nychiang, and hopefully we can merge this soon once we wrap up P6 and reporting activities. We are also gearing up for a release soon, but might delay in favor of debugging this and getting compatibility back with cc @abhyshr |
Since this change isn't necessary to build with 1.0.0, we are going to pin that version for our release, and then re-visit |
Is this ready to merge? |
Pipelines are failing, so no |
closing as this was wrapped into #84 |
update hiop interface due to the latest developments in hiop
CLOSE #28