Skip to content

Conversation

Joao-Dionisio
Copy link
Member

@Joao-Dionisio Joao-Dionisio commented Dec 9, 2024

Did as the title says, plus removed some mentions of quadratic constraints, as PySCIPOpt now covers more general constraints (following the open todo here).

Some of the documentation was wrong (getSlack, getActivity), since it incorrectly said some methods could be used with nonlinear constraints. I could adapt these functions to also cover nonlinear constraints, but it might be a bit hacky compared with what's going on on the linear side.

Draft because I don't know what should be done with these two methods.

@Opt-Mucca
Copy link
Collaborator

I like all the changes!

For the reason for it being draft: I'd ask Stefan how this is in general handled on the non-linear side for activity. I couldn't find a single function in my 30s search, but I imagine that there is one. That can be a separate MR though.

@Joao-Dionisio Joao-Dionisio marked this pull request as ready for review February 24, 2025 09:52
@Joao-Dionisio Joao-Dionisio merged commit 7e1613c into master Feb 24, 2025
1 check passed
@Joao-Dionisio Joao-Dionisio deleted the remove-quadratic branch February 24, 2025 09:56
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