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
more user friendly finite field extensions #24526
Comments
Author: Vincent Delecroix |
This comment has been minimized.
This comment has been minimized.
New commits:
|
Commit: |
Branch: u/vdelecroix/24526 |
comment:3
It may be better to keep the branch dealing with tuples/lists for speed reasons... Perhaps it doesn't matter (I don't know how much it is used or what alternatives exist for constructing extensions without too much overhead), but your patch makes the call to |
comment:4
Should this ducktyping really all be happening under the hood without notifying the user? I think a more readable error message would be better, or at least a warning. We don't want people to use SR for polynomials in their code, do we? Also, is this
necessary? The |
comment:5
Does not apply anymore. |
Work Issues: merge conflict |
comment:6
Recent user report for the same problem: |
comment:7
Please rebase, and I can review. |
GF(p).extension(xxx)
does not accept symbolic polynomials... and the error is crypticWe make it more user friendly by allowing the above in some duck typing fashion.
Component: number theory
Work Issues: merge conflict
Author: Vincent Delecroix
Branch/Commit: u/vdelecroix/24526 @
ba84546
Issue created by migration from https://trac.sagemath.org/ticket/24526
The text was updated successfully, but these errors were encountered: