ct_DEF.__test__.large_nums doctest failing in 32bit
was reported in Debian against 0.23.2 so tried blindly freshier snapshot from 0.23.x branch and that one is no go too: ====================================================================== FAIL: large_nums (line 95) (ct_DEF.__test__) Doctest: ct_DEF.__test__.large_nums (line 95) ---------------------------------------------------------------------- Traceback (most recent call last): File "/usr/lib/python2.7/doctest.py", line 2226, in runTest raise self.failureException(self.format_failure(new.getvalue())) AssertionError: Failed doctest test for ct_DEF.__test__.large_nums (line 95) File "/tmp/buildd/cython-0.23.2+git12-g2c9d175/build/work-dir/run/cpp/ct_DEF/ct_DEF.so", line unknown line number, in large_nums (line 95) ---------------------------------------------------------------------- File "/tmp/buildd/cython-0.23.2+git12-g2c9d175/build/work-dir/run/cpp/ct_DEF/ct_DEF.so", line ?, in ct_DEF.__test__.large_nums (line 95) Failed example: print_large_number(n64) Expected: -4294967295 Got: 1 -- Yaroslav O. Halchenko Center for Open Neuroscience http://centerforopenneuroscience.org Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755 Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419 WWW: http://www.linkedin.com/in/yarik
Yaroslav Halchenko schrieb am 21.09.2015 um 16:26:
was reported in Debian against 0.23.2 so tried blindly freshier snapshot from 0.23.x branch and that one is no go too:
====================================================================== FAIL: large_nums (line 95) (ct_DEF.__test__) Doctest: ct_DEF.__test__.large_nums (line 95) ---------------------------------------------------------------------- Traceback (most recent call last): File "/usr/lib/python2.7/doctest.py", line 2226, in runTest raise self.failureException(self.format_failure(new.getvalue())) AssertionError: Failed doctest test for ct_DEF.__test__.large_nums (line 95) File "/tmp/buildd/cython-0.23.2+git12-g2c9d175/build/work-dir/run/cpp/ct_DEF/ct_DEF.so", line unknown line number, in large_nums (line 95)
---------------------------------------------------------------------- File "/tmp/buildd/cython-0.23.2+git12-g2c9d175/build/work-dir/run/cpp/ct_DEF/ct_DEF.so", line ?, in ct_DEF.__test__.large_nums (line 95) Failed example: print_large_number(n64) Expected: -4294967295 Got: 1
I cannot reproduce this locally (even tried a 32bit docker container), so I blindly changed a couple of minor things in that branch. Could you give it another try? If that didn't help, here is a patch that adds a longness suffix to plain integer literals based on their type (e.g. found after coercions). I'm not sure yet if that's really a good idea, but would be helpful to know if it fixes this test. Could you try both and report back? Thanks! Stefan
On Tue, 22 Sep 2015, Stefan Behnel wrote:
Expected: -4294967295 Got: 1
I cannot reproduce this locally (even tried a 32bit docker container), so I blindly changed a couple of minor things in that branch. Could you give it another try?
If that didn't help, here is a patch that adds a longness suffix to plain integer literals based on their type (e.g. found after coercions). I'm not sure yet if that's really a good idea, but would be helpful to know if it fixes this test. Could you try both and report back?
thanks -- will do! -- Yaroslav O. Halchenko Center for Open Neuroscience http://centerforopenneuroscience.org Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755 Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419 WWW: http://www.linkedin.com/in/yarik
On Wed, 23 Sep 2015, Yaroslav Halchenko wrote:
On Tue, 22 Sep 2015, Stefan Behnel wrote:
Expected: -4294967295 Got: 1
I cannot reproduce this locally (even tried a 32bit docker container), so I blindly changed a couple of minor things in that branch. Could you give it another try?
If that didn't help, here is a patch that adds a longness suffix to plain integer literals based on their type (e.g. found after coercions). I'm not sure yet if that's really a good idea, but would be helpful to know if it fixes this test. Could you try both and report back?
thanks -- will do!
reporting: it did need guess_longness.patch to pass Thanks! -- Yaroslav O. Halchenko Center for Open Neuroscience http://centerforopenneuroscience.org Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755 Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419 WWW: http://www.linkedin.com/in/yarik
participants (2)
-
Stefan Behnel -
Yaroslav Halchenko