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

Possible bug in HeuristicDecomposition #81

Open
AndrewVSutherland opened this issue Sep 8, 2024 · 1 comment
Open

Possible bug in HeuristicDecomposition #81

AndrewVSutherland opened this issue Sep 8, 2024 · 1 comment

Comments

@AndrewVSutherland
Copy link

Possibly this is a problem in Magma or one of the packages you depend on, but if so it would be good to submit a bug in the appropriate place. This is one of several examples, which I just verified I can reproduce on Magma V2.28-11 using a fully up-to-date CHIMP install.

> C:=HyperellipticCurve(-x^5 - x^4 + 2*x^2 + 2*x + 9,x^3 + x^2);
> HeuristicDecomposition(C);
> HeuristicDecomposition(C);

In file "/home/drew/dev/magma-2.28-11/package/Geometry/RieSrf/fundamentalgroup.m", line 609, column 22:
>>         if E1`r le E2`r then
                        ^
Runtime

I note that the first call falls silently and returns nothing (but that never happens again unless you restart the Magma session).

@AndrewVSutherland AndrewVSutherland changed the title Possible bug in HeuristicSplitting Possible bug in HeuristicDecomposition Sep 8, 2024
@AndrewVSutherland
Copy link
Author

The problem goes away if you increase the precision.

> R<x> := PolynomialRing(RationalsExtra(100));
> C:=HyperellipticCurve(-x^5 - x^4 + 2*x^2 + 2*x + 9,x^3 + x^2);
> HeuristicDecomposition(C);

In file "/home/drew/dev/magma-2.28-8/package/Geometry/RieSrf/fundamentalgroup.m", line 609, column 22:
>>         if E1`r le E2`r then
                        ^
Runtime error in `: Attribute 'r' for this structure is valid but not assigned

fails but

> R<x> := PolynomialRing(RationalsExtra(200));
> C:=HyperellipticCurve(-x^5 - x^4 + 2*x^2 + 2*x + 9,x^3 + x^2);
> HeuristicDecomposition(C);
[*
Rational Field,
[*
Rational Field,
true
*], [*
[
[ 2, 1 ]
],
[]
*], [*
[
[ 2, 1 ]
],
[],
[]
*]
*]

works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant