Skip to content

Added sequential minimum optimization algorithm for SVM - #508

Merged
poyea merged 8 commits into
TheAlgorithms:masterfrom
zqbake:master
Sep 18, 2019
Merged

Added sequential minimum optimization algorithm for SVM#508
poyea merged 8 commits into
TheAlgorithms:masterfrom
zqbake:master

Conversation

@zqbake

@zqbakezqbake commented Oct 23, 2018

Copy link
Copy Markdown
Contributor

Sequential minimal optimization (SMO) is an algorithm for solving the quadratic programming (QP) problem that arises during the training of support vector machines. wiki

@zqbakezqbake changed the title Implementation of sequential minimal optimization algorithmImplementation of sequential minimal optimization algorithm for svmOct 25, 2018
@zqbake
Add demonstration of svm partition boundary
a5bf830
1:Use matplotlib show svm's partition boundary
2:Automatically download test dataset
@zqbakezqbake changed the title Implementation of sequential minimal optimization algorithm for svmAdded sequential minimum optimization algorithm for SVMOct 27, 2018
@zqbakezqbake mentioned this pull request Oct 29, 2018
Closed
@poyeapoyea self-assigned this Sep 16, 2019
@cclauss

cclauss commented Sep 16, 2019

Copy link
Copy Markdown
Member

Could we rename the file to something more helpful like sequential_minimum_optimization.py ?

cclauss
cclauss reviewed Sep 16, 2019
View reviewed changes
Comment threadmachine_learning/sequential_minimum_optimization.py Outdated
cclauss
cclauss reviewed Sep 16, 2019
View reviewed changes
Comment threadmachine_learning/sequential_minimum_optimization.py Outdated
cclauss
cclauss reviewed Sep 16, 2019
View reviewed changes
Comment threadmachine_learning/sequential_minimum_optimization.py Outdated
@zqbake
Update doc and simplify the code.
fe2832d
Fix filename typo error in doc.
Use ternary conditional operator in predict()
@zqbake
Update doc.
ac84461
@zqbake
zqbake requested a review from cclaussSeptember 17, 2019 02:39
poyea
poyea approved these changes Sep 18, 2019
View reviewed changes

@poyeapoyea left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your pull request!🤩

@poyea
poyea merged commit a2b5a90 into TheAlgorithms:masterSep 18, 2019
@zqbake

zqbake commented Sep 18, 2019

Copy link
Copy Markdown
ContributorAuthor

Thank you for your pull request!🤩

Glad to be helpful.

@onlinejudge95onlinejudge95 mentioned this pull request Sep 23, 2019
Closed
stokhos pushed a commit to stokhos/Python that referenced this pull request Jan 3, 2021
@zqbake@stokhos
Added sequential minimum optimization algorithm for SVM (TheAlgorithm…
7a18c72
…s#508)
* Implementation of sequential minimal optimization algorithm
* Update smo.py
* Add demonstration of svm partition boundary
1:Use matplotlib show svm's partition boundary
2:Automatically download test dataset
* Update smo.py
* Update smo.py
* Rename smo.py to sequential_minimum_optimization.py
* Update doc and simplify the code.
Fix filename typo error in doc.
Use ternary conditional operator in predict()
* Update doc.
@girisagar46

girisagar46 commented Oct 16, 2022

Copy link
Copy Markdown
Contributor

Hi, @zhqu1148980644

Sorry for commenting on closed PR. 🙇

In this PR, you've mentioned this link: http://web.cs.iastate.edu/~honavar/smo-svm.pdf, but the link is not working anymore. I tried to find the replacement link for it but could not find it hence, I removed it on PR #7319

Do you have the alternate link for it?

@zqbake

zqbake commented Oct 18, 2022

Copy link
Copy Markdown
ContributorAuthor

Hi, @zhqu1148980644

Sorry for commenting on closed PR. 🙇

In this PR, you've mentioned this link: http://web.cs.iastate.edu/~honavar/smo-svm.pdf, but the link is not working anymore. I tried to find the replacement link for it but could not find it hence, I removed it on PR #7319

Do you have the alternate link for it?

No.

@cclausscclauss added hacktoberfest hacktoberfest-accepted Accepted to be counted towards Hacktoberfest labels Oct 18, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@poyeapoyeapoyea approved these changes
@cclausscclaussAwaiting requested review from cclauss

Assignees

@poyeapoyea

Labels

hacktoberfesthacktoberfest-acceptedAccepted to be counted towards Hacktoberfest

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@zqbake@cclauss@girisagar46@poyea