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

# tol disables the meaning of ellipsis in doctesting #14451

Closed
cschwan mannequin opened this issue Apr 15, 2013 · 2 comments
Closed

# tol disables the meaning of ellipsis in doctesting #14451

cschwan mannequin opened this issue Apr 15, 2013 · 2 comments

Comments

@cschwan
Copy link
Mannequin

cschwan mannequin commented Apr 15, 2013

To reproduce the behavior take e.g. the doctest starting at sage/numerical/mip.pyx:2031 (p.solve() # tol 0.00001) and replace in the expected output the version string (v4.44) with the ellipsis (...). The test fails as long as it is marked with the # tol attribute. See also dicussion at https://groups.google.com/forum/#!topic/sage-devel/E2kwJLVaJx4.

Component: doctest framework

Reviewer: Jeroen Demeyer

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

@cschwan cschwan mannequin added this to the sage-5.11 milestone Apr 15, 2013
@cschwan cschwan mannequin assigned roed314 Apr 15, 2013
@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
@jdemeyer
Copy link

jdemeyer commented Oct 1, 2014

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link

jdemeyer commented Oct 1, 2014

comment:5

I agree this is a bug, however it looks quite tricky to fix. As long as the ... doesn't match any floats, it should work fine. The problem is that the code which handles floats and the code which handles ... is completely separate. I think this is too hard to get right, so I propose "wontfix".

@jdemeyer jdemeyer removed this from the sage-6.4 milestone Oct 1, 2014
@vbraun vbraun closed this as completed Oct 13, 2014
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