Opened 2 years ago

Closed 2 years ago

#26405 closed enhancement (fixed)

Root finding for p-adic polynomials

Reported by: caruso Owned by:
Priority: major Milestone: sage-8.5
Component: padics Keywords:
Cc: roed Merged in:
Authors: Xavier Caruso Reviewers: David Roe
Report Upstream: N/A Work issues:
Branch: d599b0f (Commits) Commit: d599b0f13debdb9cb6ea839c10c879780132417e
Dependencies: Stopgaps:

Description (last modified by caruso)

This ticket implements root finding for polynomials defined over any p-adic ring or field. So far, it was implemented only over Zp and Qp (using pari).

Change History (11)

comment:1 Changed 2 years ago by caruso

  • Branch set to u/caruso/root_finding_for_p_adic_polynomials

comment:2 Changed 2 years ago by git

  • Commit set to 8b79b4bb6e9c5d3c559d3f7afbecee558393f799

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

56268f0Fix stupid bug
8b79b4bDoctests

comment:3 Changed 2 years ago by git

  • Commit changed from 8b79b4bb6e9c5d3c559d3f7afbecee558393f799 to 1535dfa6945b0efb9d9668028cefd794922588c4

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

1535dfaTypo

comment:4 Changed 2 years ago by caruso

  • Authors set to Xavier Caruso
  • Description modified (diff)
  • Status changed from new to needs_review

comment:5 Changed 2 years ago by git

  • Commit changed from 1535dfa6945b0efb9d9668028cefd794922588c4 to c008a4da20f04b11f8f28483c17953bb8ec9eccb

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

c008a4dCall pari through the method factor()

comment:6 Changed 2 years ago by git

  • Commit changed from c008a4da20f04b11f8f28483c17953bb8ec9eccb to b8d38d92d54ac3b3682e9e49a52f8b01dc358c71

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

b8d38d9Merge branch 'develop' into t/26405/root_finding_for_p_adic_polynomials

comment:7 Changed 2 years ago by roed

  • Branch changed from u/caruso/root_finding_for_p_adic_polynomials to u/roed/root_finding_for_p_adic_polynomials

comment:8 Changed 2 years ago by roed

  • Commit changed from b8d38d92d54ac3b3682e9e49a52f8b01dc358c71 to d599b0f13debdb9cb6ea839c10c879780132417e

I fixed a few typos and some pyflakes errors.


New commits:

c278bf5Merge branch 'u/caruso/root_finding_for_p_adic_polynomials' of git://trac.sagemath.org/sage into t/26405/root_finding_for_p_adic_polynomials
d599b0fFixing pyflakes, fixing a few typos

comment:9 Changed 2 years ago by roed

  • Reviewers set to David Roe
  • Status changed from needs_review to positive_review

This looks good.

comment:10 Changed 2 years ago by chapoton

  • Milestone changed from sage-8.4 to sage-8.5

comment:11 Changed 2 years ago by vbraun

  • Branch changed from u/roed/root_finding_for_p_adic_polynomials to d599b0f13debdb9cb6ea839c10c879780132417e
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.