Skip to content

Get active set from solver#91

Merged
darnstrom merged 2 commits intodarnstrom:masterfrom
gianluca-garofalo:feat/get_active_set
Oct 25, 2025
Merged

Get active set from solver#91
darnstrom merged 2 commits intodarnstrom:masterfrom
gianluca-garofalo:feat/get_active_set

Conversation

@gianluca-garofalo
Copy link
Copy Markdown
Contributor

No description provided.

@darnstrom
Copy link
Copy Markdown
Owner

Thanks for this contribution!
Since it changes the result struct, I am a bit hesitant in merging (I am trying to keep the structs as slim as possible to keep the API minimal.)

The (non-weakly) active set is encoded in the dual variables that are already in the result struct. Simply checking which componentd lamda are non-zero should give you the (non-weakly) active constraints. Also note that the signs of lambda encode weather the upper or lower bound is active. A "get_active_set" function that takes in the result struct and extract the active set from the dual variables could therefore be an alternative.

But if there is a good argument for including the active set directly in the struct, I am open for merging.

Copy link
Copy Markdown
Owner

@darnstrom darnstrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a great addition to the Eigen interface! Thanks :)

@darnstrom darnstrom merged commit d2ea16c into darnstrom:master Oct 25, 2025
16 checks passed
@gianluca-garofalo gianluca-garofalo deleted the feat/get_active_set branch October 27, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants