pnnl / ExaGO

High-performance power grid optimization for stochastic, security-constrained, and multi-period ACOPF problems.
Other
64 stars 8 forks source link

meta: HiOp 1.0.1+ suport #80

Closed cameronrutherford closed 10 months ago

cameronrutherford commented 10 months ago

There are many moving parts here, and so I am creating a new issue to track all these components, and to determine the right order to operations.

Here are the current list of relevant issues (with related PRs):

We will likely need a larger PR to address all of these issues somehow, but the hope is that they can all be resolved together.

nychiang commented 10 months ago

@cameronrutherford should I try to use the latest develop branch, or release 1.6.0?

cameronrutherford commented 10 months ago

@cameronrutherford should I try to use the latest develop branch, or release 1.6.0?

We will need some patches merged into ExaGO to fix this, so definitely develop. We will then have a PR with working changes + latest HiOp running tests on all platforms

nychiang commented 10 months ago

@jaelynlitz @cameronrutherford I filed a new PR in HiOp, to withdraw -Wall and -Werror in the RELEASE mode. Therefore, I expect this can fix the compile warnings/errors you observed with unused variables.

cameronrutherford commented 10 months ago

@jaelynlitz just got back from vacation and is leading this, but that is helpful. We will generate new build with that PR and hopefully we can then re-add develop builds of hiop back with ExaGO.

We also need https://github.com/LLNL/hiop/pull/669 merged in order to support #50 and latest Ginkgo release in ExaGO, so we may need to test experimental versions of HiOp in many places at once