Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#15394 closed enhancement (fixed)

Implement a Lattes map function

Reported by: bhutz Owned by: bhutz
Priority: major Milestone: sage-6.4
Component: elliptic curves Keywords: sage-days55
Cc: defeo Merged in:
Authors: Ben Hutz Reviewers: Joao Alberto de Faria
Report Upstream: N/A Work issues:
Branch: 77ef2e9 (Commits) Commit:
Dependencies: Stopgaps:

Description

Given an elliptic curve and integer m, return the associated Lattes map on P1. It would be nice to eventually add support for non-integer m, but this would be a great first step.

Change History (11)

comment:1 in reply to: ↑ description Changed 4 years ago by pingram

I have written code for this; will submit soon-ish.

Replying to bhutz:

Given an elliptic curve and integer m, return the associated Lattes map on P1. It would be nice to eventually add support for non-integer m, but this would be a great first step.

comment:2 Changed 4 years ago by defeo

  • Cc defeo added

comment:3 Changed 4 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:4 Changed 3 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:5 Changed 3 years ago by bhutz

  • Branch set to u/bhutz/ticket/15394
  • Created changed from 11/10/13 01:05:24 to 11/10/13 01:05:24
  • Modified changed from 05/06/14 15:20:58 to 05/06/14 15:20:58

comment:6 Changed 3 years ago by bhutz

  • Authors set to Ben Hutz
  • Commit set to 52b031c4e13a2979616fc16d5a1a36e70f27e6bd
  • Status changed from new to needs_review

I've implemented this as a member function of projective space which takes E,m as parameters and returns the endomorphism. There is already functionality in elliptic curves to generate the rational expression, so the work here is only to turn that expression into an endomorphism of the appropriate projective space.


New commits:

52b031c15394: creation of Lattes map as endomorpishm of P1

comment:7 Changed 3 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:8 Changed 3 years ago by git

  • Commit changed from 52b031c4e13a2979616fc16d5a1a36e70f27e6bd to 77ef2e94fb9d706faa191817d167c70c5946a1cc

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

77ef2e915394: documentation typo fixed

comment:9 Changed 3 years ago by jdefaria

  • Reviewers set to Joao Faria
  • Status changed from needs_review to positive_review

Went through and check for bugs, found nothing outside of a small typo that has been fixed, setting to positive review

comment:10 Changed 3 years ago by vbraun

  • Branch changed from u/bhutz/ticket/15394 to 77ef2e94fb9d706faa191817d167c70c5946a1cc
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:11 Changed 3 years ago by kcrisman

  • Commit 77ef2e94fb9d706faa191817d167c70c5946a1cc deleted
  • Reviewers changed from Joao Faria to Joao Alberto de Faria
Note: See TracTickets for help on using tickets.