Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SchemeHomset_points_abelian_variety_field should be an actual group #16184

Open
sagetrac-sbesnier mannequin opened this issue Apr 18, 2014 · 3 comments
Open

SchemeHomset_points_abelian_variety_field should be an actual group #16184

sagetrac-sbesnier mannequin opened this issue Apr 18, 2014 · 3 comments

Comments

@sagetrac-sbesnier
Copy link
Mannequin

sagetrac-sbesnier mannequin commented Apr 18, 2014

As discussed in #11474 , SchemeHomset_points_abelian_variety_field is not really a group, but it should be !

However, note that the class of the points of an elliptic curve inherits from AdditiveGroupElement.

CC: @defeo @JohnCremona @pjbruin

Component: elliptic curves

Author: Sébastien Besnier

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

@sagetrac-sbesnier sagetrac-sbesnier mannequin added this to the sage-6.2 milestone Apr 18, 2014
@sagetrac-sbesnier sagetrac-sbesnier mannequin self-assigned this Apr 18, 2014
@defeo

This comment has been minimized.

@defeo
Copy link
Member

defeo commented Apr 18, 2014

comment:2

Please set the component field of your tickets.

@pjbruin
Copy link
Contributor

pjbruin commented Apr 22, 2014

comment:3

Note that there are two distinct but related ways in which objects of type SchemeHomset_points_abelian_variety_field "are not groups":

  • the class SchemeHomset_points_abelian_variety_field itself does not inherit from sage.groups.abelian_gps.abelian_group.AbelianGroup_class;
  • if A is an instance of SchemeHomset_points_abelian_variety_field, the list of categories returned by A.categories() does not include "Category of commutative groups" (i.e. the category obtained as Groups().Commutative()), let alone "Category of groups" (i.e. Groups()).
    Both points are probably easy to fix, but it would also be good to have an example of (a) concrete new feature(s) that this would introduce.

@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants