Testing for RAJAHIOPSPARSE equality and inequality constraint Jacobians#31
Merged
pelesh merged 10 commits intoApr 15, 2026
Merged
Conversation
…ian) + bug fix for PBPOL. Co-authored-by: William A Perkins <william.perkins@pnnl.gov>
db4f6b4 to
50a4b41
Compare
Collaborator
|
I am getting test failures for 9-bus and 200-bus systems. IEEE 118-bus works fine. Full output for the 9-bus system: Details |
smarter way to verify the stacked inequality constraints.
Collaborator
Author
Yes I see it as well. It comes from the inequality constraint Jacobian being stacked after the equality constraints ones. Working on a fix. |
pelesh
approved these changes
Apr 15, 2026
Collaborator
pelesh
left a comment
There was a problem hiding this comment.
All tests pass on multiple platforms.
nkoukpaizan
added a commit
that referenced
this pull request
May 28, 2026
…ns (#31) * test_acopf with PBPOLRAJAHIOPSPARSE (verify equality constraint Jacobian) + bug fix for PBPOL. Co-authored-by: William A Perkins <william.perkins@pnnl.gov> * Cleanup changes to RAJAHIOPSPARSE nnz computation. * Workaround for RAJAHIOPSPARSE inequality constraint verification. * Handling cases where IPOPT or HIOP are not available in test_acopt + bug fixes. * HiOp-compatible checks for constraint Jacobian allocations. Will need a smarter way to verify the stacked inequality constraints. * Fix for inequality constraint offsets. --------- Co-authored-by: William A Perkins <william.perkins@pnnl.gov> Co-authored-by: nkoukpaizan <nkoukpaizan@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge request type
Relates to
This MR updates
Summary
This adds a way to test equality and inequality constraint Jacobians with RAJAHIOPSPARSE models.
Still need to do the same thing for the hessians.