Ticket #12204 (needs_work enhancement)

Opened 18 months ago

Last modified 18 months ago

Reducing the defining polynomial of a hyperelliptic curve

Reported by: florian Owned by: was
Priority: major Milestone: sage-5.11
Component: number theory Keywords: hyperelliptic curves sd35
Cc: mstreng Work issues:
Report Upstream: N/A Reviewers: Marco Streng
Authors: Florian Bouyer Merged in:
Dependencies: #6341 Stopgaps:

Description (last modified by florian) (diff)

A method for hyperellipt curves over rational field which reduces the defining polynomial

Attachments

trac_12204_reduce_poly.patch Download (1.9 KB) - added by florian 18 months ago.
trac_12204_modify_doc.patch Download (1.8 KB) - added by florian 18 months ago.

Change History

Changed 18 months ago by florian

comment:1 Changed 18 months ago by florian

  • Status changed from new to needs_review

comment:2 Changed 18 months ago by florian

  • Authors set to Florian Bouyer

comment:3 Changed 18 months ago by florian

  • Description modified (diff)

comment:4 Changed 18 months ago by mstreng

  • Status changed from needs_review to needs_work
  • Reviewers set to Marco Streng
  1. The output of this new method is another curve over QQ, which is isomorphic over QQbar, but possibly not isomorphic over QQ. That is not clear from the documentation.
  2. Also, why is it called "reduce_poly" and not simply "reduce"?
  3. Related to 1. it would be great to also have a function that gives something isomorphic over QQ, and in that case also an isomorphism defined over QQ. This could be done with the same function if you make it (for example) "def reduce(self, rational=True, isomorphism=False)".

I'd say 1. is required for a positive review. 3 may be something for a later ticket if it is too much work.

comment:5 Changed 18 months ago by florian

Created ticket #12209 for number 3.

It is called reduce_poly as there is already a general scheme method called reduce which 

"Return the corresponding reduced algebraic space associated to this scheme."

Changed 18 months ago by florian

comment:6 Changed 18 months ago by florian

  • Description modified (diff)

I have changed the documentation

Note: See TracTickets for help on using tickets.