-
-
Notifications
You must be signed in to change notification settings - Fork 503
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
make accessing coefficients of finite-field elements easier #33748
Comments
Reviewer: Travis Scrimshaw |
comment:2
Should we also add an While it is not in the built documentation, if this ever gets moved into something public, the |
Branch pushed to git repo; I updated commit sha1. New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:5
Thanks — indeed, |
comment:6
Thanks. LGTM. |
Changed branch from public/access_finite_field_element_coefficients to |
This comes up rather frequently:
For number fields, we do have a shorthand:
This patch adds a
.__getitem__()
method to the classFinitePolyExtElement
, which makesa[i]
andlist(a)
andtuple(a)
work as expected.For compatibility with number-field elements, we also add
.list()
, which should behave identically to callinglist()
on the element.Component: algebra
Author: Lorenz Panny
Branch/Commit:
121d0ab
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/33748
The text was updated successfully, but these errors were encountered: