[nkoukpaizan]

Merge request type

• New feature
• Resolves bug
• Documentation
• Other

Relates to

• OPFLOW
• SOPFLOW
• SCOPFLOW
• TCOPFLOW
• CMake build system
• Spack configuration
• Manual
• Web docs
• Other

This MR updates

• Header files
• Source code
• CMake build system
• Spack configuration
• Web docs
• Manual
• Other

Summary

This saves the number of iterations from the solvers (uniformly named as number_iterations per comments on #46).

This is a partial, isolated, fix from the nicholson/testing-cmake-scopflow-fix branch, used in recent Frontier runs. #66 already contributed the changes needed to run larger number of contingencies. DCOPF workaround is the remaining component. cc @tsybinae

I left one change (HIOP iterations counts) commented out to minimize the diffs. Uncommenting if will require updating the test values. We may remove the checks for number of iterations altogether.

Linked Issue(s)

Closes #42 by adding an all reduce of the convergence status.

[pelesh]

Checking number of iterations to converged solution was used as a (poor) replacement for derivatives (Jacobian and Hessian) unit tests. It has been poorly designed and highly unreliable. I suggest we remove these checks altogether because they add little value and significant maintenance overhead.

Instead, I would focus on completing Hessian unit tests, which are the last remaining test that need to be implemented.