Opened 9 years ago

Closed 3 years ago

Last modified 3 years ago

#11806 closed enhancement (fixed)

Optional spkg for MPFRCX

Reported by: jpflori Owned by: tbd
Priority: major Milestone: sage-8.1
Component: packages: optional Keywords: mpfrcx multiprecision.org complex real univariate polynomial ecc2011
Cc: leif, nbruin Merged in:
Authors: Jean-Pierre Flori Reviewers: Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: 71152b9 (Commits) Commit:
Dependencies: Stopgaps:

Description (last modified by jpflori)

Include the MPFRCX library (​http://mpfrcx.multiprecision.org) as an optional spkg.

Upstream tarball at:

Change History (15)

comment:1 Changed 9 years ago by leif

  • Cc leif added

comment:2 Changed 9 years ago by zimmerma

  • Keywords ecc2011 added

comment:3 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:4 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:5 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:6 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:7 Changed 3 years ago by jpflori

  • Branch set to u/jpflori/mpfrcx
  • Cc nbruin added
  • Commit set to be4e5382048824adafc23a946b6c805cf91cc1cf
  • Description modified (diff)
  • Status changed from new to needs_review

I'll make a thin wrapper as another optional package asap.

New commits:

​be4e538Add optional MPFRCX package.

comment:8 Changed 3 years ago by tmonteil

It seems you did a typo when filling the "Dependencies" field of the ticket (a very old closed ticket).

comment:9 Changed 3 years ago by jdemeyer

  • Dependencies #8664 deleted
  • Milestone changed from sage-6.4 to sage-8.1
  • Reviewers set to Jeroen Demeyer
  • Status changed from needs_review to needs_work

I'm curious what you plan to use this for, but the package is fine.

One detail: there are long lines in SPKG.txt which should be wrapped to < 80 chars. If you fix this, you can set the ticket to positive_review.

comment:10 Changed 3 years ago by jpflori

Make a small wrapper to get polynomials over RR and CC and have an efficient complex multiprecision FFT.

comment:11 Changed 3 years ago by git

  • Commit changed from be4e5382048824adafc23a946b6c805cf91cc1cf to 71152b95055de7ce72dc95c4a180897880edb553

Branch pushed to git repo; I updated commit sha1. New commits:

​71152b9Break long lines.

comment:12 Changed 3 years ago by jpflori

  • Status changed from needs_work to needs_review

comment:13 Changed 3 years ago by jdemeyer

  • Status changed from needs_review to positive_review

comment:14 Changed 3 years ago by vbraun

  • Branch changed from u/jpflori/mpfrcx to 71152b95055de7ce72dc95c4a180897880edb553
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:15 Changed 3 years ago by jdemeyer

  • Commit 71152b95055de7ce72dc95c4a180897880edb553 deleted

This was closed but the tarball isn't mirrored.

Note: See TracTickets for help on using tickets.