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.

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

GenericBackend: Add _test_sense method #21499

Closed
mkoeppe opened this issue Sep 15, 2016 · 9 comments
Closed

GenericBackend: Add _test_sense method #21499

mkoeppe opened this issue Sep 15, 2016 · 9 comments
Labels
c: linear programming c: numerical p: major / 3 t: enhancement
Milestone
sage-7.5

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 15, 2016

This adds another test method, which all MIP backends pass.

CC: @dimpase

Component: numerical

Author: Matthias Koeppe

Branch/Commit: a3a9a0f

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/21499
@mkoeppe mkoeppe added this to the sage-7.4 milestone Sep 15, 2016
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Sep 15, 2016

Branch: u/mkoeppe/add_test_sense_method

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Sep 15, 2016

Commit: a3a9a0f

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Sep 15, 2016

New commits:

a3a9a0fGenericBackend._test_sense: New

@dimpase
Copy link
Member

dimpase commented Sep 15, 2016

comment:3

We'll be away for 5 days, for a vacation...

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Sep 15, 2016

comment:4

Enjoy your vacation!
This is not urgent.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Nov 20, 2016

comment:5

Still needs review...

@dimpase
Copy link
Member

dimpase commented Nov 20, 2016

Reviewer: Dima Pasechnik

@dimpase
Copy link
Member

dimpase commented Nov 20, 2016

comment:6

A lot of tests fail if I test with Nonexistent_LP_solver in --optional.
Should all these tests be tagged differently?
(another option would be to add a backend so named :-))

@dimpase dimpase modified the milestones: sage-7.4, sage-7.5 Nov 20, 2016
@vbraun
Copy link
Member

vbraun commented Nov 21, 2016

Changed branch from u/mkoeppe/add_test_sense_method to a3a9a0f

@vbraun vbraun closed this as completed in 1469a37 Nov 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: linear programming c: numerical p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-7.5
Development

No branches or pull requests
3 participants
@vbraun @dimpase @mkoeppe