-
Notifications
You must be signed in to change notification settings - Fork 14
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 in Clustering_MLJ caused by spelling mistake #65
Comments
Unless you already have them, can I suggest adding MLJ interface tests? See MLJTestInterface.jl and this example in MLJXGBoostInterface.jl tests. I expect these would pick these spelling mistakes up. |
Hi.. all BetaML - MLJ model interfaces have tests, however these are "manual" and focus on the various specific task of the model (classification, regression, etc...) in these cases the typo was on a one-line function ( For new interfaces I will surely use the specific MLJ test framework.. |
Thank you. |
Need
fitesult
->fitresult
inBetaML.jl/src/Clustering/Clustering_MLJ.jl
Line 178 in df11d62
See also #64
The text was updated successfully, but these errors were encountered: