Opened 4 years ago
Last modified 4 years ago
#17476 new task
Improve polynomial templates
Reported by: | jdemeyer | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-6.5 |
Component: | basic arithmetic | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
The files src/sage/rings/polynomial/polynomial_template.pxi
and src/sage/libs/flint/nmod_poly_linkage.pxi
are a mess:
- They are almost completely undocumented
- The functions in it cannot be interrupted (see for example #17470)
celement_pow
should really be split up in 2 functions: one for the modular case, one for normal powering- This shouldn't be needed:
except OverflowError: return Polynomial.__pow__(self, ee, modulus)
Change History (2)
comment:1 Changed 4 years ago by
- Summary changed from Polynomial templates are a mess to Improve polynomial templates
comment:2 Changed 4 years ago by
Note: See
TracTickets for help on using
tickets.
We should surely have a look at the files organization for p-adics templates as these were written much more recently and possibly in a cleaner way.