#17852: Small cleanup in rings.arith and rings.integer
-------------------------------------+-------------------------------------
Reporter: vdelecroix | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.6
Component: basic arithmetic | Resolution:
Keywords: | Merged in:
Authors: Vincent Delecroix | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/vdelecroix/17852 | 975c2a0a6563538377eca78590fd16ddf0342548
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by vdelecroix):
* status: needs_work => needs_review
Old description:
> In this ticket:
> - we implement the two functions `prime_part_m` and `binomial` directly
> as methods of Sage integers and get rid of the `late_import` in
> `sage.rings.integer`
> - simplify the import logic in `rings.arith`
> - replace in arith `prod([x for x in hum])` by `prod(x for x in hum)`
> - remove some useless comments in arith
New description:
In this ticket:
- we implement the two functions `prime_part_m` and `binomial` directly
as methods of Sage integers and get rid of the `late_import` in
`sage.rings.integer`
- simplify the import logic in `rings.arith`
- replace in arith `prod([x for x in hum])` by `prod(x for x in hum)`
- remove some useless comments in arith
- deprecate `powermodm_ui`
--
Comment:
Replying to [comment:10 jdemeyer]:
> What's the point of `eratosthenes()`, shouldn't that be a
`deprecated_function_alias` of `prime_range`?
It is excplicitely written "for educational purpose"... I do not want to
deprecate it so roughly.
I agreed with all of the other suggestions and implemented them.
Vincent
--
Ticket URL: <http://trac.sagemath.org/ticket/17852#comment:20>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
