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.

Sign up for GitHub

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

Jump to bottom

parameter allow_subfield of absolute_order_from_module_generators has no effect #16046

Closed
saraedum opened this issue Apr 2, 2014 · 5 comments
Closed

parameter allow_subfield of absolute_order_from_module_generators has no effect #16046

saraedum opened this issue Apr 2, 2014 · 5 comments
Labels
c: number theory r: wontfix t: bug

Comments

@saraedum
Copy link
Member

saraedum commented Apr 2, 2014

The following doctest in sage.rings.number_field.order.absolute_order_from_module_generators is incorrect:

sage: F.<alpha> = NumberField(x**4+3)
sage: F.order([alpha**2], allow_subfield=True)
Order in Number Field in alpha with defining polynomial x^4 + 3

This should be a order in a smaller field. The code for allow_subfield seems incomplete.

CC: @sagetrac-jakobkroeker

Component: number theory

Stopgaps: wrongAnswerMarker

Reviewer: Julian Rüth

Issue created by migration from https://trac.sagemath.org/ticket/16046
@saraedum saraedum added this to the sage-6.2 milestone Apr 2, 2014
@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
@sagetrac-jakobkroeker
Copy link
Mannequin

sagetrac-jakobkroeker mannequin commented Mar 4, 2017

Stopgaps: wrongAnswerMarker

@saraedum
Copy link
Member Author

saraedum commented Mar 9, 2018

comment:4

Interesting. I just ran into this problem again. We never added the stopgap actually.

@saraedum
Copy link
Member Author

saraedum commented Mar 9, 2018

comment:5

Fixed in #24934.

@saraedum saraedum removed this from the sage-6.4 milestone Mar 9, 2018
@saraedum
Copy link
Member Author

saraedum commented Mar 9, 2018

Reviewer: Julian Rüth

@videlec
Copy link
Contributor

videlec commented May 18, 2018

comment:7

closing positively reviewed duplicates

@videlec videlec closed this as completed May 18, 2018
@saraedum saraedum mentioned this issue Jan 29, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: number theory r: wontfix t: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@saraedum @videlec