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

Reduce or avoid GPL dependencies #130

Open
andreasnoack opened this issue Oct 9, 2023 · 6 comments
Open

Reduce or avoid GPL dependencies #130

andreasnoack opened this issue Oct 9, 2023 · 6 comments

Comments

@andreasnoack
Copy link

In version 2.2 a dependency on Copulas was introduced. This pulls in GSL and crlibm which are both GPL licensed. The package also has a direct dependency on FFTW which is also GPL so it would be useful to switch to AbstractFFTs or a less restrictive alternative to allow using this package in a non-GPL context.

@ChrisRackauckas
Copy link
Member

We can make it an extension

@devmotion
Copy link
Member

it would be useful to switch to AbstractFFTs

I quickly looked into it and it seems it would require JuliaMath/AbstractFFTs.jl#56.

@andreasnoack
Copy link
Author

The GSL dependency in Copulas looks like it was silently removed in lrnv/Copulas.jl#73 so if the version bound here is updated the part of the issue here disappears.

@Vaibhavdixit02
Copy link
Member

Copulas seems to have had some breaking changes in patch releases and it wasn't immediately clear to me how to fix the error in https://github.com/SciML/GlobalSensitivity.jl/actions/runs/8392054624/job/22983804323?pr=134#step:6:434 so I have opened an issue at the repo lrnv/Copulas.jl#186 will followup and create a release with the compact bounds bumped once it becomes clear how to rewrite the usage here.

@devmotion
Copy link
Member

The GSL/Copula issue was fixed by #134, thank you @Vaibhavdixit02!

@Vaibhavdixit02
Copy link
Member

Reopening since the FFTW issue still exists.

@Vaibhavdixit02 Vaibhavdixit02 reopened this Apr 5, 2024
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

4 participants