-
Notifications
You must be signed in to change notification settings - Fork 569
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
Patch of Dierckx_jll causing errors in Dierckx.jl #10266
Comments
Well, it's hard to predict that an upstream non-breaking version would break things, isn't it? |
There isn't much we can do in Yggdrasil about this. You have two options basically: work with upstream Dierckx to fix the problem and get a new version we can build, or introduce a compat entry for the jll to filter out the broken version in the Dierckx.jl package. |
Sounds like some arguments were changed: scipy/scipy@4fcc237#diff-38ac20ea49e75f06240b089eef3a03d932773a30d5b96128741f5d2f2c4188db |
If there were argument changes, that's a breaking change. Shouldn't this have been a v0.x minor release instead of a patch? If this were v0.2, it would have required people to check for compat instead of automatically updating. |
👆 |
I guess the scipy people don't consider that to be public since it is their underlying Fortran code. The other option is going to the horses mouth and just getting stuff from netlib directly, but they don't seem to have an all-encompasing tar file for the dierckx library, just individual routines. |
Dierckx_jll recently release v0.1.1. As a patch release, that should be non-breaking, but there are several errors occurring in Dierckx.jl that are fixed by downgrading Dierckx_jll back to v0.1.0:
JuliaMath/Dierckx.jl#98
JuliaMath/Dierckx.jl#99
The text was updated successfully, but these errors were encountered: