#15185: Clean up interface to the PARI library
-------------------------------------+-------------------------------------
Reporter: pbruin | Owner:
Type: enhancement | Status: new
Priority: major | Milestone: sage-6.0
Component: interfaces | Resolution:
Keywords: pari | Merged in:
Authors: Peter Bruin | Reviewers:
Report Upstream: N/A | Work issues: rebase
Branch: u/pbruin/15185 | Commit:
Dependencies: #9640, #10018, | 2344f0bd14b5f72eb84ad1b134f84a74de25fc2e
#11868 | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by pbruin):
* dependencies: #9640, #10018 => #9640, #10018, #11868
* work_issues: wait for #10018 => rebase
Old description:
> The file `sage/libs/pari/gen.pyx` is too big and contains too many
> different things. For clarity and maintainability, it should be split
> into two or three parts:
> - `gen.pyx` containing the class `gen`;
> - `pari_instance.pyx` containing the class `PariInstance` and some
> utility functions;
> - (maybe) `pari_error.pyx` containing the class `PariError`.
> In addition, this ticket makes the following changes:
> - The `gen` class currently contains duplicates of various methods of
> `PariInstance` (such as `new_gen`) for no obvious reason. These methods
> will be deleted.
> - Replace the global variables `t0, ..., t5` (used to store results of
> conversion of Python objects to PARI objects) by appropriate local
> variables.
> - Remove obsolete code (e.g. `add_unsafe`, which currently is not used,
> and probably never should be).
New description:
The file `sage/libs/pari/gen.pyx` is too big and contains too many
different things. For clarity and maintainability, it should be split
into two or three parts:
- `gen.pyx` containing the class `gen`;
- `pari_instance.pyx` containing the class `PariInstance` and some utility
functions;
- (maybe) `pari_error.pyx` containing the class `PariError`.
In addition, this ticket makes the following changes:
- The `gen` class currently contains duplicates of various methods of
`PariInstance` (such as `new_gen`) for no obvious reason. These methods
will be deleted.
- Remove obsolete code (e.g. `add_unsafe`, which currently is not used,
and probably never should be).
--
Comment:
As a first step I split off the changes eliminating the global GEN
variables as a patch on #11868. This will certainly have to be rebased,
but that had to be done anyway because of #10018.
--
Ticket URL: <http://trac.sagemath.org/ticket/15185#comment:9>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.
