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
Gap interface for number fields #9423
Comments
Fixing two bugs (doctested) in the GAP interface of number fields |
comment:1
Attachment: trac_9423_gap_for_numberfields.patch.gz |
comment:2
The code corrects a couple of bugs in the gap interface of number fields. Since ! cannot be part of the name of a generator of a number field, then eliminating "!" from the gap representation is correct. The solution to the "E" variable problem is correct, althought there should be a more system-wide solution to this kind of problems. I will not give it a positive review until #5618 is also ready to merge, since this patch eliminates a doctest that after #5618 will be obsolete. |
Reviewer: Luis Felipe Tabera Alonso |
Attachment: trac_9423_gap_for_numberfields.2.patch.gz Updated headers |
Merged: sage-4.6.2.alpha2 |
Originally motivated by work on #5618, I found two bugs in the Gap interface for number fields, reported here.
#8909 has a positive review and seems partially relevant here, so, I started work with the patch from #8909 applied.
With the new patch, the following works (and is doctested):
Fixing the second problem, it is needed to avoid a conflict with an internal variable name of a GAP function, namely "E". This tests that the conflict is indeed avoided:
Component: interfaces
Keywords: gap interface number field
Author: Simon King
Reviewer: Luis Felipe Tabera Alonso
Merged: sage-4.6.2.alpha2
Issue created by migration from https://trac.sagemath.org/ticket/9423
The text was updated successfully, but these errors were encountered: