Ticket #701 (closed enhancement: fixed)

Opened 6 years ago

Last modified 6 years ago

[with patch] port srange to Cython for speed

Reported by: malb Owned by: somebody
Priority: major Milestone: sage-2.8.6
Component: basic arithmetic Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

William complained about srange being slow several times now. Let's fix it.

Attachments

srange.hg Download (4.3 KB) - added by robertwb 6 years ago.
srange-fixes.hg Download (2.5 KB) - added by robertwb 6 years ago.

Change History

Changed 6 years ago by robertwb

comment:1 Changed 6 years ago by was

  • Status changed from new to closed
  • Resolution set to fixed

comment:2 Changed 6 years ago by was

  • Milestone changed from sage-2.9 to sage-2.8.4.3

comment:3 Changed 6 years ago by jsp

  • Status changed from closed to reopened
  • Resolution fixed deleted

The srange function with include_endpoint=True does not include the endpoint in some cases:

sage: srange(1.0, 5.0, include_endpoint=True)

[1.00000000000000, 2.00000000000000, 3.00000000000000, 4.00000000000000]

Changed 6 years ago by robertwb

comment:4 Changed 6 years ago by was

  • Summary changed from port srange to Cython for speed to port srange to Cython for speed [patch attached]
  • Milestone changed from sage-2.8.5 to sage-2.8.6

comment:5 Changed 6 years ago by mhansen

  • Summary changed from port srange to Cython for speed [patch attached] to [with patch] port srange to Cython for speed

comment:6 Changed 6 years ago by was

  • Status changed from reopened to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.