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

Add FLINT declarations for finite fields and p-adics #19646

Closed
jdemeyer opened this issue Nov 29, 2015 · 17 comments
Closed

Add FLINT declarations for finite fields and p-adics #19646

jdemeyer opened this issue Nov 29, 2015 · 17 comments

Comments

@jdemeyer
Copy link

Add the declarations for #14304 and #16664.

CC: @roed314 @jpflori @saraedum

Component: interfaces

Author: Julian Rueth, David Roe, Jean-Pierre Flori, Jeroen Demeyer

Branch: 6cc4151

Reviewer: Jean-Pierre Flori

Issue created by migration from https://trac.sagemath.org/ticket/19646

@jdemeyer jdemeyer added this to the sage-6.10 milestone Nov 29, 2015
@jdemeyer
Copy link
Author

Branch: u/jdemeyer/ticket/19646

@jdemeyer
Copy link
Author

New commits:

75bd5ecAdd FLINT declarations for finite fields and p-adics

@jdemeyer
Copy link
Author

Commit: 75bd5ec

@jdemeyer
Copy link
Author

comment:3

This ticket is mostly just copying stuff from #14304 and #16664. I did add a few more changes, so somebody should still look at the branch.

@jpflori
Copy link

jpflori commented Nov 30, 2015

comment:4

Thanks for splitting this out of the other tickets.
A note on the fq bindings, could you add the fq_div functions?
They were mysteriously left out of the release when I started #16664, but should be in now.

@jdemeyer
Copy link
Author

comment:5

Replying to @jpflori:

Thanks for splitting this out of the other tickets.
A note on the fq bindings, could you add the fq_div functions?
They were mysteriously left out of the release when I started #16664, but should be in now.

I saw them commented out, so I didn't know what to do with them.

Note that there was no need to comment them out in the first place: it's not a problem to declare functions which don't actually exist. You will get a compiler error when you use them, but not if you just declare them.

@jdemeyer
Copy link
Author

comment:6

There is still no fq_nmod_div...

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 30, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

2fa8600Add fq_div()

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 30, 2015

Changed commit from 75bd5ec to 2fa8600

@jpflori
Copy link

jpflori commented Nov 30, 2015

comment:8

Yes indeed...
And I see

FLINT_DLL void fq_div(fq_t rop, const fq_t op1, const fq_t op2, const fq_ctx_t ctx);
FLINT_DLL void fq_sqr(fq_t rop, const fq_t op, const fq_ctx_t ctx);
FLINT_DLL void fq_div(fq_t rop, const fq_t op1, const fq_t op2, const fq_ctx_t ctx);
FLINT_DLL void fq_sqr(fq_t rop, const fq_t op, const fq_ctx_t ctx);

in the trunk version of fq.h.
I'll make a pull request upstream.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 30, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

6cc4151Use standard copyright template

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 30, 2015

Changed commit from 2fa8600 to 6cc4151

@jpflori
Copy link

jpflori commented Dec 1, 2015

Reviewer: Jean-Pierre Flori

@jdemeyer
Copy link
Author

jdemeyer commented Dec 1, 2015

comment:11

Thanks!

@vbraun
Copy link
Member

vbraun commented Dec 2, 2015

Changed branch from u/jdemeyer/ticket/19646 to 6cc4151

@jpflori
Copy link

jpflori commented Dec 3, 2015

Changed commit from 6cc4151 to none

@jpflori
Copy link

jpflori commented Dec 3, 2015

comment:13

Actually there is a fq_nmod_div prototype which gets defined through the inclusion of fq_templates.h.

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

No branches or pull requests

3 participants