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

Bring Doctest coverage for integer_mod.pyx to 100% #12260

Open
roed314 opened this issue Jan 4, 2012 · 7 comments
Open

Bring Doctest coverage for integer_mod.pyx to 100% #12260

roed314 opened this issue Jan 4, 2012 · 7 comments
Assignees
@sagetrac-mvngu
Labels
p: major / 3 pending s: needs work t: enhancement t: tests

Comments

@roed314
Copy link
Contributor

roed314 commented Jan 4, 2012

Part of metaticket #12024

Depends on #12048
Depends on #12039

Component: doctest coverage

Author: David Roe

Issue created by migration from https://trac.sagemath.org/ticket/12260
@roed314 roed314 added this to the sage-5.4 milestone Jan 4, 2012
@roed314
Copy link
Contributor Author

roed314 commented Jan 4, 2012

Attachment: 12260.patch.gz

@roed314
Copy link
Contributor Author

roed314 commented Jan 4, 2012

Dependencies: #12039

@roed314
Copy link
Contributor Author

roed314 commented Jan 4, 2012

Author: David Roe

@roed314
Copy link
Contributor Author

roed314 commented Jan 4, 2012

comment:1

The dependency on #12039 is because that ticket fixes a pickling bug in sage.structure.parent.Set_PythonType_class.

@jdemeyer
Copy link

jdemeyer commented Apr 6, 2012

Changed dependencies from #12039 to #12048, #12039

@kcrisman
Copy link
Member

comment:3

Is there any way you can make this not depend on #12039, or is that essential for (say) doing a loads(dumps(X)) test? Currently #12039 does not apply.

@jdemeyer jdemeyer removed this from the sage-5.4 milestone Oct 5, 2012
@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Nov 27, 2013

comment:5

Patch does not apply. And depends on two tickets without patch O_o

Nathann

@williamstein williamstein mentioned this issue Feb 22, 2013
Closed
@loefflerd loefflerd mannequin mentioned this issue Mar 18, 2012
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-mvngu sagetrac-mvngu

Labels
p: major / 3 pending s: needs work t: enhancement t: tests
Projects
None yet
Milestone
No milestone
Development

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