Skip to content

Add farkas dual for variable bounds #363

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 9 commits into from
Nov 5, 2020
Merged

Add farkas dual for variable bounds #363

merged 9 commits into from
Nov 5, 2020

Conversation

@odow
Copy link
Member

@odow odow commented Oct 28, 2020 •
edited
Loading

cc @juan-pablo-vielma

image

TODO:

  • Are the signs correct for min/max?
  • Are the signs correct for EqualTo and Interval?

mlubin
mlubin reviewed Oct 28, 2020
View reviewed changes
@juan-pablo-vielma
Copy link

juan-pablo-vielma commented Oct 28, 2020

the GRBgetvars version works on my example too.

joaquimg
joaquimg reviewed Oct 28, 2020
View reviewed changes
@odow odow requested a review from blegat October 28, 2020 22:09
odow
odow commented Oct 28, 2020
View reviewed changes
@joaquimg joaquimg mentioned this pull request Oct 28, 2020
Closed
@odow
Copy link
Member Author

odow commented Oct 29, 2020

image

@blegat
Copy link
Member

blegat commented Oct 29, 2020 •
edited
Loading

Note that you could use MOI.Utilities.get_fallback to get the duals of variable bounds computed from other duals.

@odow odow mentioned this pull request Nov 3, 2020
Merged
odow
odow commented Nov 4, 2020
View reviewed changes
@odow odow merged commit 797c90f into master Nov 5, 2020
@odow odow deleted the od/farkas branch November 5, 2020 19:43
@odow odow mentioned this pull request Nov 5, 2020
Merged
@odow odow mentioned this pull request Nov 17, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mlubin mlubin mlubin left review comments

@joaquimg joaquimg joaquimg left review comments

@blegat blegat Awaiting requested review from blegat

+1 more reviewer

@juan-pablo-vielma juan-pablo-vielma juan-pablo-vielma left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@odow @juan-pablo-vielma @blegat @mlubin @joaquimg