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

I would like to re-use Primes.Factorization for polynomials #126

Closed
jmichel7 opened this issue Jan 26, 2023 · 10 comments
Closed

I would like to re-use Primes.Factorization for polynomials #126

jmichel7 opened this issue Jan 26, 2023 · 10 comments

Comments

@jmichel7
Copy link
Contributor

jmichel7 commented Jan 26, 2023

I have written code for factoring a univariate polynomial over a finite field, and over the integers (modules Fact and FFfac in my package Gapjm). I thought it would be convenient to use Primes.Factorization to store these factorizations. However, I cannot since it is written in several places T<:Integer.

Are these annotations really useful? It seems to me they are not...

@oscardssmith
Copy link
Member

I think this should be doable. I don't believe we are using the assumption of integer type anywhere.

@jmichel7
Copy link
Contributor Author

Excellent! I wait for this... Unless you want me to make a pr?

@oscardssmith
Copy link
Member

can you make the PR? I'll merge.

@jmichel7
Copy link
Contributor Author

jmichel7 commented Jul 3, 2023

Thank you for merging! Are you going to do a release? It would be good to have this and also the function divisors ...

@oscardssmith
Copy link
Member

I'm going to give myself a few days (mainly because I might want to get #120 in also). that said if I haven't made a release in a week or so, feel free to bump this.

as for divisors, I'm probably not going to write the code myself, but if there was a PR for it I probably would merge (even though I kind of think the multiplicative functions might belong in a separate package)

@jmichel7
Copy link
Contributor Author

jmichel7 commented Jul 3, 2023

But the code for divisors is already there in master!

@oscardssmith
Copy link
Member

oscardssmith commented Jul 3, 2023

oh, that will be very easy then :)

@jmichel7
Copy link
Contributor Author

"if I haven't made a release in a week or so, feel free to bump this."

BUMP!

@oscardssmith
Copy link
Member

version tagged. sorry for the wait.

@fingolfin
Copy link
Contributor

Can this be closed?

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

No branches or pull requests

3 participants