sagemath / sage

Main repository of SageMath
https://www.sagemath.org
Other
1.36k stars 462 forks source link

Weil restriction for elliptic curves over number fields #13266

Open 7834196f-58c7-4da2-b8f5-addd43f0a866 opened 12 years ago

7834196f-58c7-4da2-b8f5-addd43f0a866 commented 12 years ago

This enhancement takes an elliptic curve, finds its defining ideal and then applies the Weil restriction.

Component: elliptic curves

Keywords: weil restriction

Author: Hester Graves

Issue created by migration from https://trac.sagemath.org/ticket/13266

7834196f-58c7-4da2-b8f5-addd43f0a866 commented 12 years ago

This is the code and documentation of a propsed enhancement.

kini commented 12 years ago

Description changed:

--- 
+++ 
@@ -1 +1 @@
-This enhancement takes an elliptic curve, finds its defining ideal and then applies the Weil restriction.  
+This enhancement takes an elliptic curve, finds its defining ideal and then applies the Weil restriction.
kini commented 12 years ago
comment:1

Attachment: newweil13266.gz

I take it Hester Graves is your (the patch author's) name? I've put it in the Author field instead of the Summary field, and put a summary of the code change in the Summary field (I hope it's accurate, I don't know anything about elliptic curves).

kini commented 12 years ago

Author: Hester Graves

kini commented 12 years ago

apply to $SAGE_ROOT/devel/sage

kini commented 12 years ago
comment:2

Attachment: newweil13266.patch.gz

I reuploaded your patch with a filename ending in .patch so that it gets a nice diff display on trac, and also so that the patchbot notices it.

JohnCremona commented 12 years ago
comment:3

Question: Is this supposed to be only for quadratic fields, as suggest by "abelian surface"? If so, it needs to be said i nthe documentation. If not, the function name needs changing, and an exaple of a higher degree field included.

nbruin commented 12 years ago
comment:4

Another question: The current implementation makes

E.weilrestriction()

an alias for

E.defining_ideal().weilrestriction()

so it saves typing 17 characters, at the expense of implementing "weilrestriction" for elliptic curves as a functor from the category of elliptic curves over a number field K to the category of ideals over Q.

I'm afraid most people would expect the weil restriction of an elliptic curve to be an abelian variety, not an ideal.

Plus, in what sense is the returned ideal describing the Weil restriction?

Since you're restricting a projective ideal, you have to be careful what you divide out by. Projective equivalence for (x:y:z) in P^2(K) is modulo K^* whereas for (x0:x1:y0:y1:z0:z1) in P^5(Q) it's only modulo Q^*.

You also run into problems because there are subvarieties where (x0,x1,y0,y1,z0,z1) are not all zero and yet correspond to (x,y,z) with no x,y,z invertible (if K is a field over Q then these varieties won't have Q-rational points, though), so you'd have to take something much more complicated than Proj(...), and something that depends on the field extension that you have forgotten by now, before this gives you the Weil restriction.

There is a reason why Weil restrictions are taken of affine varieties and then glued together.

I'm afraid that saving 17 keystrokes is not worth the misleading terminology. This routine needs a fundamentally different approach before it can be considered for inclusion.