pnnl / ExaGO

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

Update Summit build scripts with new module dependencies. #149

Open pelesh opened 4 days ago

pelesh commented 4 days ago

Merge request type

Relates to

This MR updates

Summary

Script loading modules with ExaGO dependencies on Summit has been out of date. This PR updates build scripts to point to current modules available on Summit built by @nkoukpaizan.

Linked Issue(s)

nkoukpaizan commented 1 day ago

This will need a bit of work for the develop branch. Some of the module I built are older versions of the dependencies (e.g., HiOp v0.7.2; the interface has changed since).

pelesh commented 1 day ago

This will need a bit of work for the develop branch. Some of the module I built are older versions of the dependencies (e.g., HiOp v0.7.2; the interface has changed since).

Is there any major dependency other than HiOp where the new version changes the API? If not, I would just remove HiOp module from the script, since in most use cases one would want to build their own HiOp. There is no CI pipeline using these modules so the testing would work as well as it works now.

nkoukpaizan commented 1 day ago

Yeah, the script currently doesn't load the HiOp module, which is fine when providing our own HiOp build. But this breaks the default ExaGo buildsystem (i.e. running MY_CLUSTER=summit ./buildsystem/build.sh). Merging it as is might come back to bite us later.