Opened 15 years ago
Closed 15 years ago
#1704 closed enhancement (fixed)
[with patch, with positive review] replace _DivPolyContext by _multiply_point
Reported by: | David Harvey | Owned by: | William Stein |
---|---|---|---|
Priority: | major | Milestone: | sage-2.10.1 |
Component: | algebraic geometry | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
This patch replaces the _DivPolyContext
class with a new function _multiply_point
. The main downside of the original _DivPolyContext
is that it's very recursive, and I started overflowing python's stack for some large problems I needed to play with. The new function is not recursive, and also turns out to be slightly faster.
Attachments (1)
Change History (4)
Changed 15 years ago by
Attachment: | multiply_point.hg added |
---|
comment:1 Changed 15 years ago by
comment:2 Changed 15 years ago by
Summary: | [with patch] replace _DivPolyContext by _multiply_point → [with patch, with positive review] replace _DivPolyContext by _multiply_point |
---|
comment:3 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Merged in Sage 2.10.1.alpha1
Note: See
TracTickets for help on using
tickets.
I can't speak to mathematical correctness, but the patch looks good to me. Apply.