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

make more integer factorization algorithms available to user #14970

Closed
mstreng opened this issue Jul 25, 2013 · 11 comments
Closed

make more integer factorization algorithms available to user #14970

mstreng opened this issue Jul 25, 2013 · 11 comments
Labels
c: number theory p: major / 3 t: enhancement
Milestone
sage-6.1

Comments

@mstreng
Copy link
Contributor

mstreng commented Jul 25, 2013

This is a matter of adding an "algorithm" option to the function "factor" and wrapping some of the functionality that is currently not easily accessible.

The patch may conflict with #1145, but that ticket has been immobile for a long time.

CC: @aghitza

Component: number theory

Keywords: sd51

Author: Alex Ghitza

Branch/Commit: u/ncohen/14970 @ 510d554

Reviewer: Nathann Cohen

Issue created by migration from https://trac.sagemath.org/ticket/14970
@mstreng
Copy link
Contributor Author

mstreng commented Jul 25, 2013

Changed keywords from none to sd51

@aghitza
Copy link
Contributor

aghitza commented Jul 26, 2013

comment:2

Attachment: trac_14970-factor.patch.gz

Patch attached; it only makes the qsieve and ecm-gmp functionality more readily available from factor.

@aghitza
Copy link
Contributor

aghitza commented Jul 26, 2013

Author: AlexGhitza

@aghitza aghitza added this to the sage-5.12 milestone Jul 26, 2013
@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Dec 2, 2013

Dependencies: u/ncohen/14970

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Dec 2, 2013

comment:3

Okay, it makes sense :-)

Here is the same patch as a git branch. I just removed a couple of trailing whitespaces.

Nathann

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Dec 2, 2013

Reviewer: Nathann Cohen

@jdemeyer
Copy link
Contributor

jdemeyer commented Dec 2, 2013

Changed author from AlexGhitza to Alex Ghitza

@jdemeyer
Copy link
Contributor

jdemeyer commented Dec 2, 2013

Branch: u/ncohen/14970

@jdemeyer
Copy link
Contributor

jdemeyer commented Dec 2, 2013

Commit: 510d554

@jdemeyer
Copy link
Contributor

jdemeyer commented Dec 2, 2013

New commits:

[510d554](https://github.com/sagemath/sagetrac-mirror/commit/510d554)trac #14970: make qsieve and ecm-gmp easily available from factor

@jdemeyer
Copy link
Contributor

jdemeyer commented Dec 2, 2013

Changed dependencies from u/ncohen/14970 to none

@jdemeyer jdemeyer modified the milestones: sage-5.13, sage-6.0 Dec 2, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.0, sage-6.1 Dec 17, 2013
@vbraun vbraun closed this as completed in 0ccec0c Dec 22, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: number theory p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-6.1
Development

No branches or pull requests
4 participants
@aghitza @vbraun @jdemeyer @mstreng