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

Fix CoinBackend testsuite _test_solve on 32 bit #21550

Open
jdemeyer opened this issue Sep 21, 2016 · 8 comments
Open

Fix CoinBackend testsuite _test_solve on 32 bit #21550

jdemeyer opened this issue Sep 21, 2016 · 8 comments

Comments

@jdemeyer
Copy link

On 32-bit Linux (Linux arando 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:05:16 UTC 2016 i686 i686 i686 GNU/Linux) with Sage 7.4.beta4:

sage: from sage.numerical.backends.generic_backend import get_solver
sage: p = get_solver(solver="Coin")
sage: TestSuite(p).run()
Failure in _test_pickling:
Traceback (most recent call last):
  File "/home/jdemeyer/sage-git/local/lib/python2.7/site-packages/sage/misc/sage_unittest.py", line 283, in run
    test_method(tester = tester)
  File "sage/structure/sage_object.pyx", line 620, in sage.structure.sage_object.SageObject._test_pickling (/home/jdemeyer/sage-git/src/build/cythonized/sage/structure/sage_object.c:5233)
    tester.assertEqual(loads(dumps(self)), self)
  File "/home/jdemeyer/sage-git/local/lib/python/unittest/case.py", line 515, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/home/jdemeyer/sage-git/local/lib/python/unittest/case.py", line 508, in _baseAssertEqual
    raise self.failureException(msg)
AssertionError: <type 'sage.numerical.backends.coin_backend.CoinBackend'> != <type 'sage.numerical.backends.coin_backend.CoinBackend'>
------------------------------------------------------------
Failure in _test_solve:
Traceback (most recent call last):
  File "/home/jdemeyer/sage-git/local/lib/python2.7/site-packages/sage/misc/sage_unittest.py", line 283, in run
    test_method(tester = tester)
  File "sage/numerical/backends/generic_backend.pyx", line 697, in sage.numerical.backends.generic_backend.GenericBackend._test_solve (/home/jdemeyer/sage-git/src/build/cythonized/sage/numerical/backends/generic_backend.c:8748)
    with tester.assertRaises(MIPSolverException) as cm:   # unbounded
  File "/home/jdemeyer/sage-git/local/lib/python/unittest/case.py", line 116, in __exit__
    "{0} not raised".format(exc_name))
AssertionError: MIPSolverException not raised
------------------------------------------------------------
The following tests failed: _test_pickling, _test_solve

The _test_pickling failure happens also on 64-bit systems (pickling is not supported by the MIP backends), but the _test_solve failure only on 32-bit.

CC: @mkoeppe @sagetrac-tmonteil

Component: numerical

Keywords: sdl

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

@jdemeyer jdemeyer added this to the sage-7.5 milestone Sep 21, 2016
@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 21, 2016

comment:5

Pickling is not supported by the MIP backends.
Is there a better way of declaring this than just disabling the _test_pickling test?

@jdemeyer
Copy link
Author

comment:6

Replying to @mkoeppe:

Is there a better way of declaring this than just disabling the _test_pickling test?

I think that's the best way because it's very explicit.

@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 22, 2016

comment:7

Thanks. I've updated title and description.

@mkoeppe mkoeppe changed the title Fix CoinBackend testsuite Fix CoinBackend testsuite _test_solve on 32 bit Sep 22, 2016
@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Aug 27, 2019

Changed keywords from none to sdl

@mkoeppe mkoeppe removed this from the sage-7.5 milestone Dec 29, 2022
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

2 participants