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
[fixed by #5842] constructing the ring of integers of a relative number field is SLOW #5136
Comments
comment:1
Actually, CTRL-C does eventually stop this, with the following result:
|
comment:2
It is, in fact, not an infinite loop. It's just really, really slow. It also seems to pre-date 3.3.alpha3:
|
comment:3
I do not think this is a problem with the relative number field patch; it was an existing problem. I have code that I can submit if you want to compute the relative ring of integers; it uses pari's rnfbasis. This issue has to with the fact that |
comment:4
This works for me now (in sage 4.0.2):
I don't know what's changed -- presumably this is something to do with Francis Clarke's campaign to fix all the relative number field bugs over the last couple of months -- but presumably we can close this ticket now? |
comment:6
Replying to @loefflerd:
Yes, this issue had already been raised in #4738, and I commented there that "The problem of the slowness of computing relative maximal orders is solved by the patch in #5842. A doctest is included at line 532 of the patched What changed was a rewrite of |
comment:7
We should close this ticket -- two people agree that it's fixed and there are doctests to prove it. |
comment:8
I'm setting this to "positive review" so the release manager can close it when convenient. |
comment:9
I'm closing this ticket as a "duplicate." |
David Loeffler reported this on sage-devel:
Note also that CTRL-C seems to have no effect. I see the same problem on my 32-bit Archlinux machine.
Component: number fields
Issue created by migration from https://trac.sagemath.org/ticket/5136
The text was updated successfully, but these errors were encountered: