Ticket #6200 (closed enhancement: fixed)

Opened 9 months ago

Last modified 8 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 9 months ago.
trac_6200-review.patch Download (2.2 KB) - added by mhansen 9 months ago.

Change History

Changed 9 months ago by fredrik.johansson

Changed 9 months ago by fredrik.johansson

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

Changed 9 months ago by mhansen

Changed 9 months ago by mhansen

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

Changed 9 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 8 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.