#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 | 101f5a0feaf66f5f5e3a7cb0e4d706220ee3ff71
Dependencies: #16878 | Stopgaps:
-------------------------------------+-------------------------------------
Description changed by jdemeyer:
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
> - deprecate `powermodm_ui`
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`
- remove 1 from the output of `prime_powers()` (consistent with #16878)
--
--
Ticket URL: <http://trac.sagemath.org/ticket/17852#comment:35>
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.
