Opened 6 years ago

Closed 6 years ago

#20798 closed defect (fixed)

some bugs in padic BSD

Reported by: wuthrich Owned by:
Priority: major Milestone: sage-7.3
Component: elliptic curves Keywords: padic BSD, padic height
Cc: Merged in:
Authors: Chris Wuthrich Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: 43c5a59 (Commits, GitHub, GitLab) Commit: 43c5a59c186a72b3f1c5912a31bd016ba2023335
Dependencies: Stopgaps:

Status badges


Bernadette Perrin-Riou reports two bugs:

One is that this does not work

sage: E = EllipticCurve("91b")
sage: h = E.padic_height(7,10)

The other is that the 2-adic L-series takes gamma to be 1+2 rather than 1+4.

Change History (8)

comment:1 Changed 6 years ago by chapoton

  • Branch set to public/20798
  • Commit set to d20cdced744ec90be1791323973eb1b769cebff2

Here is first step, for the padic_height.

I am not able to know what to do for gamma.

New commits:

d20cdcetrac 20798 using padic_height instead of height

comment:2 Changed 6 years ago by wuthrich

That was quick. I did that part myself already and I am working on the other.

comment:3 Changed 6 years ago by wuthrich

  • Branch changed from public/20798 to u/wuthrich/ticket/20798
  • Commit changed from d20cdced744ec90be1791323973eb1b769cebff2 to 43c5a59c186a72b3f1c5912a31bd016ba2023335

I modified the code for the p-adic L-functions, both ordinary and supersingular. The error bounds should be correct.

comment:4 Changed 6 years ago by wuthrich

  • Status changed from new to needs_review

comment:5 Changed 6 years ago by chapoton

  • Authors set to Chris Wuthrich
  • Reviewers set to Frédéric Chapoton

for the bot to run, you need to fill the Author field

comment:6 Changed 6 years ago by wuthrich

good to know, thanks.

comment:7 Changed 6 years ago by chapoton

  • Status changed from needs_review to positive_review

ok, looks good to me

comment:8 Changed 6 years ago by vbraun

  • Branch changed from u/wuthrich/ticket/20798 to 43c5a59c186a72b3f1c5912a31bd016ba2023335
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.