Skip to content
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.

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

Inequalities in MixedIntegerLinearProgram are not properly handled #12546

Closed
ppurka opened this issue Feb 20, 2012 · 7 comments
Closed

Inequalities in MixedIntegerLinearProgram are not properly handled #12546

ppurka opened this issue Feb 20, 2012 · 7 comments

Comments

@ppurka
Copy link
Member

ppurka commented Feb 20, 2012

The <= and >= inequalities are not properly handled in MixedIntegerLinearProgram - essentially in both sage.numerical.mip.LinearConstraint and sage.numerical.mip.LinearFunction. Here is an example:

sage: p = MixedIntegerLinearProgram()
sage: b = p.new_variable()
sage: b[0] <= b[1] <= 2  # This is not ok
x_1 <= 2
sage: (b[0] <= b[1] <= 2).constraints # Not ok
[x_1, 2]
sage: b[0] >= b[1] # This is ok
x_1 <= x_0
sage: 1 >= b[1]    # This is not since it starts with a constant
False
sage: 1 >= b[1] >= 2*b[0] # Not ok
False
sage: b[2] >= b[1] >= 2*b[0]  # Not ok
2 x_0 <= x_1

CC: @dimpase

Component: linear programming

Keywords: MixedIntegerLinearProgram LinearConstraint LinearFunction

Reviewer: Dmitrii Pasechnik, Punarbasu Purkayastha

Issue created by migration from https://trac.sagemath.org/ticket/12546

@dimpase
Copy link
Member

dimpase commented Feb 20, 2012

comment:1

this is a duplicate of #12091, right?

@ppurka
Copy link
Member Author

ppurka commented Feb 20, 2012

comment:2

This is dupe of #12091. Please close. I don't see the option to close tickets. ( I got no powers :'( )

@dimpase
Copy link
Member

dimpase commented Feb 20, 2012

comment:3

Replying to @ppurka:

This is dupe of #12091. Please close. I don't see the option to close tickets. ( I got no powers :'( )

Me neither, but we can positively review it to its quick and painless demise.

Hoogedelgeleerde Release Manager, AUB, dit is een dupe...

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Feb 20, 2012

comment:4

This + "positive review" is the standard incantation :-)

Nathann

@nathanncohen nathanncohen mannequin removed this from the sage-5.0 milestone Feb 20, 2012
@dimpase
Copy link
Member

dimpase commented Feb 20, 2012

comment:5

Replying to @nathanncohen:

This + "positive review" is the standard incantation :-)

Nathann

natuurlijk, als u nu in Brussel woont, moet u ook de laatste woord, "dupe", ofwel "slachtoffer", kennen :)

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Feb 20, 2012

comment:6

natuurlijk, als u nu in Brussel woont, moet u ook de laatste woord, "dupe", ofwel "slachtoffer", kennen :)

My god. This last incantation is not in my book. Hopefully Jeroen will understand :-D

Nathann

@jdemeyer
Copy link

Reviewer: Dmitrii Pasechnik, Punarbasu Purkayastha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants