Skip to content
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

Implement more type conversions #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Implement more type conversions #9

wants to merge 2 commits into from

Conversation

fingolfin
Copy link
Member

@fingolfin fingolfin commented Jul 24, 2014

This pull request is work in progress, DO NOT MERGE!

We currently only really support polynomials and rings in characteristic 0 when it comes to conversion (and even that is lacking). There is not yet much support for working over finite fields, not even over prime fields.

As long as one only works with Singular objects, and never needs to convert them back to GAP objects, that may be fine. But on the long run, we really need to provide conversion methods for as many of these as possible; at least for all finite fields and for Z/mZ.

Back in January, I picked Hans' brain on this, and collected sufficient information to get started on converting Singular "number" objects (i.e. coefficients). Sadly, beyond taking some notes and some code skeletons, I did not get around to actually implementing anything.

I put my notes for that into this branch, in the hope that I or somebody else can pick up this work later on. (Of course you'll then want to start a new branch or rebase this one, instead of keeping the nonsense commit).

Beyond coefficient conversions, we are also in dire need of functions which easily convert between GAP polynomials and Singular polynomials (clearly, in general this requires coefficient conversions to work, but even for rational coefficients the situation is not great). I'll add some incomplete prototypes of that, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant