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

Galois group when polynomial has too high degree #13810

Closed
jm58660 mannequin opened this issue Dec 8, 2012 · 18 comments
Closed

Galois group when polynomial has too high degree #13810

jm58660 mannequin opened this issue Dec 8, 2012 · 18 comments

Comments

@jm58660
Copy link
Mannequin

jm58660 mannequin commented Dec 8, 2012

In 5.4, with KASH installed:

NumberField(x^21-2, 'a').galois_group(type="pari").order()

gives answer 252, but with

NumberField(x^22-2, 'a').galois_group(type="pari").order()

I got

Error, the variable '_s_' must have a value

I guess this relates to kash interface.

Component: number fields

Author: Jori Mäntysalo

Branch/Commit: a755205

Reviewer: Frédéric Chapoton

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

@jm58660 jm58660 mannequin added this to the sage-5.11 milestone Dec 8, 2012
@jm58660 jm58660 mannequin added c: number fields labels Dec 8, 2012
@jm58660 jm58660 mannequin assigned loefflerd Dec 8, 2012
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jul 4, 2013

comment:1

Tested with 5.10: After trying R.=QQ[]; (x^23-2).galois_group().order() I can't calculate for example R.=QQ[]; (x^20-2).galois_group().order(). It seems that internal(?) kash instance fails and Sage don't notice it. User must restart worksheet to get this running again.

@fchapoton

This comment has been minimized.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Oct 30, 2013

comment:4

In 5.13beta1 with kash package

NumberField(x^15-2, 'a').galois_group(type="pari").order()

gives

ValueError: Index n must be in {1,..,None}

@jdemeyer jdemeyer removed this from the sage-6.1 milestone Jan 9, 2014
@jdemeyer

This comment has been minimized.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 14, 2014

comment:7

With sage-6.3 this is slightly more buggy: Trying to with 23. degreee polynomial sage says "Kash crashed -- automatically restarting." but system does not restart kash.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Sep 29, 2014

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Sep 29, 2014

Commit: cbc8acd

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Sep 29, 2014

comment:9

There has been some other modifications on src/sage/rings/polynomial/polynomial_rational_flint.pyx that conflicts with this patch.

In any case, http://page.math.tu-berlin.de/~kant/.3a6ca5.html clearly says that polynomials up to degree 23 are supported. But just changing given example x_O:=MaximalOrder(X^2+5); Galois(x_O); to have x^23+5 gives segfault and degree 22 get stuck. So I suggests trying KASH only when degree <= 21.


New commits:

cbc8acdChanged limit of using KASH from <=23 to <=21.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 2, 2015

Changed commit from cbc8acd to f9ab830

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 2, 2015

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

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 2, 2015

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

a755205Changed the limit from 23 to 21.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 2, 2015

Changed commit from f9ab830 to a755205

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jun 2, 2015

comment:12

As also the web page http://page.math.tu-berlin.de/~kant/kash.html has been changed to "polynomials over Q up to degree 21" (from 23), I think this is clearly the right thing to do.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jun 2, 2015

Author: Jori Mäntysalo

@jm58660 jm58660 mannequin added this to the sage-6.8 milestone Jun 2, 2015
@jm58660 jm58660 mannequin added p: minor / 4 and removed wishlist item labels Jun 2, 2015
@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jun 2, 2015

comment:13

I am getting tired of your adding me in Cc of your ticket every single time you want a review.

Nathann

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Jun 3, 2015

comment:14

Replying to @nathanncohen:

I am getting tired of your adding me in Cc of your ticket every single time you want a review.

OK, I'll stop doing that. Sorry.

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@vbraun
Copy link
Member

vbraun commented Jun 6, 2015

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