#23235 closed enhancement (fixed)

Fast p-adic exponential

Reported by: caruso Owned by:
Priority: major Milestone: sage-8.0
Component: padics Keywords: padic exponential, sd87
Cc: roed, saraedum, jpflori Merged in:
Authors: Xavier Caruso Reviewers: Adele Bourgeois
Report Upstream: N/A Work issues:
Branch: 594ac63 (Commits) Commit: 594ac634dbb5607c66d84a8c3c353ca1d9bcd313
Dependencies: Stopgaps:

Description

This ticket provides a C implementation of two fast algorithms that compute p-adic exponential.

Change History (11)

comment:1 Changed 18 months ago by caruso

  • Branch set to u/caruso/padicexp

comment:2 Changed 18 months ago by git

  • Commit set to 72b5f9d369c75d5c80319d3188a757bf9abb3cc7

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

4946dddNewton scheme for solving the equation log(x) = a
72b5f9dBinary splitting algorithm and optimized Newton algorithm over Qp

comment:3 Changed 18 months ago by git

  • Commit changed from 72b5f9d369c75d5c80319d3188a757bf9abb3cc7 to 138933863737b77cbe2cb3f3e6d618a66eff74c3

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

1389338Added doctests

comment:4 Changed 18 months ago by caruso

  • Authors set to Xavier Caruso
  • Status changed from new to needs_review

I think that this ticket is now ready for review...

comment:5 Changed 18 months ago by roed

Awesome! I'm headed up to the mountains for a week or so; if saraedum hasn't reviewed it by the time I'm back, I'll take a look.

comment:6 Changed 18 months ago by jpflori

  • Cc jpflori added

comment:7 Changed 18 months ago by git

  • Commit changed from 138933863737b77cbe2cb3f3e6d618a66eff74c3 to 42b5472bf048e70ea45db22ab8f7dc36cc2aac7e

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

42b5472Merge branch 'develop' into padicexp

comment:8 Changed 18 months ago by git

  • Commit changed from 42b5472bf048e70ea45db22ab8f7dc36cc2aac7e to 594ac634dbb5607c66d84a8c3c353ca1d9bcd313

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

594ac63Small fixes

comment:9 Changed 17 months ago by roed

  • Keywords sd87 added

comment:10 Changed 17 months ago by abourgeois

  • Reviewers set to Adele Bourgeois
  • Status changed from needs_review to positive_review

All tests pass!

comment:11 Changed 17 months ago by vbraun

  • Branch changed from u/caruso/padicexp to 594ac634dbb5607c66d84a8c3c353ca1d9bcd313
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.