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

doctest issue -- combining # long and # 32-bit / # 64-bit #2235

Closed
craigcitro opened this issue Feb 20, 2008 · 9 comments
Closed

doctest issue -- combining # long and # 32-bit / # 64-bit #2235

craigcitro opened this issue Feb 20, 2008 · 9 comments
Labels
c: doctest framework p: minor / 4 t: bug
Milestone
sage-5.9

Comments

@craigcitro
Copy link
Member

Combining # long with # 32-bit / # 64-bit seems to completely ignore the # long directive.

There's a long doctest in sage/rings/number_field/totallyreal_rel.py that is what caused me to notice this; I've made that a # no doctest for now, because it was causing timeouts on some machines. When this bug is fixed, that needs to be changed into a # long.

Depends on #12415

CC: @ncalexan

Component: doctest framework

Author: Jeroen Demeyer

Reviewer: David Roe

Merged: sage-5.9.beta0

Issue created by migration from https://trac.sagemath.org/ticket/2235
@craigcitro craigcitro added this to the sage-5.8 milestone Feb 20, 2008
@craigcitro
Copy link
Member Author

comment:1

Note: there's a long doctest in sage/rings/number_field/totallyreal_rel.py that is what caused me to notice this; I've made that a # no doctest for now, because it was causing timeouts on some machines. When this bug is fixed, that needs to be changed into a # long.

@jdemeyer

This comment has been minimized.

Sign in to view
@jdemeyer
Copy link

jdemeyer commented Mar 6, 2013

Author: Jeroen Demeyer

@jdemeyer
Copy link

jdemeyer commented Mar 7, 2013

comment:3

Just need to add a doctest that this works now.

@jdemeyer
Copy link

comment:4

Attachment: 2235_long_time.patch.gz

@jdemeyer
Copy link

Dependencies: #12415

@roed314
Copy link
Contributor

roed314 commented Mar 14, 2013

Reviewer: David Roe

@roed314
Copy link
Contributor

roed314 commented Mar 14, 2013

comment:6

Looks good to me.

@jdemeyer jdemeyer modified the milestones: sage-5.8, sage-5.9 Mar 14, 2013
@jdemeyer
Copy link

Merged: sage-5.9.beta0

@jdemeyer jdemeyer mentioned this issue Sep 10, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: doctest framework p: minor / 4 t: bug
Projects
None yet
Milestone
sage-5.9
Development

No branches or pull requests
3 participants
@roed314 @craigcitro @jdemeyer