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
Newton SVM #409
Newton SVM #409
Conversation
|
||
/** | ||
* set C | ||
* @param C constant C |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrong indention
Harshit, looks like very nice work now! Please do the few final formatting fixes - mostly indention is still wrong / whitspace fixes and ifdefs need to be at beginning of lines and then this can be merged! |
okay soeren, will commit new changes in a while, And on contypebinary_splice dataset it takes around 6-14 sec to train on whole dataset, Time varies on amount of memory available i guess. |
I was having issues with removing conflicts in my previous pull request, So now I have created a new pull request in which everything is resolved,
I have rebased upstream/master in this, So i don't think there will be any more conflicts now.