-
Notifications
You must be signed in to change notification settings - Fork 33
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
Bug? #98
Comments
barucden
added a commit
to barucden/LIBSVM.jl
that referenced
this issue
Jan 28, 2024
We were missing the fields `nu` and `regularize_bias` in the `Parameter` structure. Fixes JuliaML#98.
It is a bug in LIBLINEAR.jl. Structure It would help if LIBLINEAR.jl is in JuliaML (JuliaML/LIBLINEAR.jl#27). |
barucden
added a commit
to barucden/LIBSVM.jl
that referenced
this issue
Apr 19, 2024
I verified that the added test errors with LIBLINEARv0.6. With LIBLINEARv0.7, the test passes. Curiously, LIBLINEAR is only used when fitting LinearSVC through the "scikit" API (never through `svmtrain`/`svmpredict`). Resolves JuliaML#98
barucden
added a commit
that referenced
this issue
Apr 19, 2024
I verified that the added test errors with LIBLINEARv0.6. With LIBLINEARv0.7, the test passes. Curiously, LIBLINEAR is only used when fitting LinearSVC through the "scikit" API (never through `svmtrain`/`svmpredict`). Resolves #98
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems that the MLJ interface for this package has stopped working, without any change to the code: JuliaAI/MLJLIBSVMInterface.jl#18
The culprit appears to be the linear SVC model. Locally testing this package reveals no problem, but it looks to me that there are no tests for the linear case. A MWE that reproduces the same fail observed in the MLJ interface package is below:
All my testing is with julia v1.10.0 and LIBSVM v0.8.0.
Since the last release of LIBSVM.jl, there was an update to the C library, LIBLINEAR. From their website: "Version 2.47 released on July 9, 2023. We fix some minor bugs." Could this be the culprit? Be good if someone else can confirm this or otherwise resolve the issue 😄.
The text was updated successfully, but these errors were encountered: