ddbourgin / numpy-ml

Machine learning, in numpy
https://numpy-ml.readthedocs.io/
GNU General Public License v3.0
15.35k stars 3.72k forks source link

add svm basic algorithms, not passed test #37

Closed daidai21 closed 5 years ago

ddbourgin commented 5 years ago

Hi @daidai21 - I just took a look this PR. I see that the code is copied directly from https://github.com/LasseRegin/SVM-w-SMO/blob/master/SVM.py .

This is unacceptable.

PRs you submit should reflect earnest attempts at implementing a model yourself. To be absolutely clear: it is fine to reference others' code. It is not fine to blindly copy it without attribution -- that is plagiarism.

I appreciate that you are eager to contribute to the repo. Trust me, I want you to contribute. The key word here is you -- a pull request with a wrapper around someone else's code is not a contribution from you.

If you still want to work on this, I suggest first trying to understand the mathematics behind the model, then work towards your own implementation.