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:

Status badges

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)

multiply_point.hg (2.5 KB) - added by David Harvey 15 years ago.

Download all attachments as: .zip

Change History (4)

Changed 15 years ago by David Harvey

Attachment: multiply_point.hg added

comment:1 Changed 15 years ago by ncalexan

I can't speak to mathematical correctness, but the patch looks good to me. Apply.

comment:2 Changed 15 years ago by ncalexan

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 Michael Abshoff

Resolution: fixed
Status: newclosed

Merged in Sage 2.10.1.alpha1

Note: See TracTickets for help on using tickets.