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:

  1. They are almost completely undocumented
  2. The functions in it cannot be interrupted (see for example #17470)
  3. celement_pow should really be split up in 2 functions: one for the modular case, one for normal powering
  4. This shouldn't be needed:
            except OverflowError:
                return Polynomial.__pow__(self, ee, modulus)
    

Change History (2)

comment:1 Changed 4 years ago by jdemeyer

  • Summary changed from Polynomial templates are a mess to Improve polynomial templates

comment:2 Changed 4 years ago by jpflori

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.

Note: See TracTickets for help on using tickets.