GlobalSensitivity.jl icon indicating copy to clipboard operation
GlobalSensitivity.jl copied to clipboard

Reduce or avoid GPL dependencies

Open andreasnoack opened this issue 2 years ago • 6 comments

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.

andreasnoack avatar Oct 09 '23 20:10 andreasnoack

We can make it an extension

ChrisRackauckas avatar Oct 09 '23 20:10 ChrisRackauckas

it would be useful to switch to AbstractFFTs

I quickly looked into it and it seems it would require https://github.com/JuliaMath/AbstractFFTs.jl/issues/56.

devmotion avatar Oct 10 '23 08:10 devmotion

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

andreasnoack avatar Mar 22 '24 11:03 andreasnoack

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 https://github.com/lrnv/Copulas.jl/issues/186 will followup and create a release with the compact bounds bumped once it becomes clear how to rewrite the usage here.

Vaibhavdixit02 avatar Mar 22 '24 18:03 Vaibhavdixit02

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

devmotion avatar Apr 04 '24 07:04 devmotion

Reopening since the FFTW issue still exists.

Vaibhavdixit02 avatar Apr 05 '24 05:04 Vaibhavdixit02