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

process chained equalities with different signs correctly #13696

Closed
dimpase opened this issue Nov 10, 2012 · 6 comments
Closed

process chained equalities with different signs correctly #13696

dimpase opened this issue Nov 10, 2012 · 6 comments

Comments

@dimpase
Copy link
Member

dimpase commented Nov 10, 2012

we have to deal with what remained from #12091. At least, report errors for the inputs like the following:

sage: p = MixedIntegerLinearProgram()
sage: b = p.new_variable()
sage: b[0] <= 555*b[1] >= 2
sage: b[0] <= 555*b[1] == 2

which even with #12091 are incorrectly processed

sage: b[0] <= 555*b[1] >= 2
2 <= x_0 <= 555*x_1
sage: b[0] <= 555*b[1] == 2
555*x_1 == 2

Depends on #14540

CC: @vbraun @ppurka @nathanncohen

Component: linear programming

Reviewer: Jeroen Demeyer

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

@dimpase dimpase added this to the sage-5.11 milestone Nov 10, 2012
@dimpase dimpase added the t: bug label Nov 10, 2012
@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 3, 2016

comment:6

Description modified to make the example self-contained.

@mkoeppe

This comment has been minimized.

@dimpase dimpase removed this from the sage-6.4 milestone Apr 5, 2016
@jdemeyer
Copy link

Changed dependencies from #12091 to #14540

@jdemeyer jdemeyer added this to the sage-7.2 milestone Apr 19, 2016
@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 20, 2016

comment:10

And how about:

sage: b[0] <= 3 <= 4 <= b[1]
4 <= x_1

@jdemeyer
Copy link

Reviewer: Jeroen Demeyer

@jdemeyer jdemeyer removed this from the sage-7.2 milestone Apr 21, 2016
@jdemeyer
Copy link

comment:13

The reviewer of #14540 was not happy with a partial solution, so it's probably better to thoroughly fix the problem on #14540.

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

4 participants