Ticket #6200 (closed enhancement: fixed)

Opened 15 months ago

Last modified 14 months ago

[with patch, positive review] Use mpmath to compute constants

Reported by: fredrik.johansson Owned by: jkantor
Priority: major Milestone: sage-4.1
Component: numerical Keywords:
Cc: Author(s): Fredrik Johannson, Mike Hansen
Report Upstream: Reviewer(s): Nick Alexander
Merged in: sage-4.1.rc0 Work issues:

Description

Assumes that mpmath has been added to Sage (#6196)

Patch summary:

Use mpmath to compute numerical values of constants.

Previously khinchin, mertens and twinprime were LimitedPrecisionConstant?, but with this patch they can be computed to any precision.

The patch also adds the Glaisher constant (which is available in mpmath) and corrects the name of the Mertens constant.

Attachments

constants.patch Download (14.9 KB) - added by fredrik.johansson 15 months ago.
trac_6200-review.patch Download (2.2 KB) - added by mhansen 15 months ago.

Change History

Changed 15 months ago by fredrik.johansson

Changed 15 months ago by fredrik.johansson

  • summary changed from Use mpmath to compute constants to [with patch; needs review] Use mpmath to compute constants

Changed 15 months ago by mhansen

Changed 15 months ago by mhansen

For backward compatibility, I added merten as an alias to mertens. Other than that, the patch looks good.

Changed 15 months ago by ncalexan

  • reviewer set to Nick Alexander
  • milestone set to sage-4.0.3
  • summary changed from [with patch; needs review] Use mpmath to compute constants to [with patch, positive review] Use mpmath to compute constants
  • author set to Fredrik Johannson, Mike Hansen

Looks good to me. Apply both patches.

Changed 14 months ago by rlm

  • status changed from new to closed
  • resolution set to fixed
  • merged set to sage-4.1.rc0
Note: See TracTickets for help on using tickets.