Skip to content

Added example to sklearn.svm.NuSVC#26972

Closed
TheCheshireCat6 wants to merge 5 commits intoscikit-learn:mainfrom
TheCheshireCat6:sklworkshop
Closed

Added example to sklearn.svm.NuSVC#26972
TheCheshireCat6 wants to merge 5 commits intoscikit-learn:mainfrom
TheCheshireCat6:sklworkshop

Conversation

@TheCheshireCat6
Copy link
Copy Markdown

Issue #26927

Added example plot_svm_nonlinear.py to sklearn.svm.NuSVC
@adrinjalali @glemaitre

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 1, 2023

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 96cdf9d. Link to the linter CI: here

Comment on lines +1090 to +1091
See :ref:`sphx_glr_auto_examples_svm_plot_svm_nonlinear.py`
for a more detailed example.
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.

I think plot_svm_nonlinear.py could also be removed, we have the other example that @StefanieSenger is working on which nicely includes non linear kernels. WDYT @glemaitre

Copy link
Copy Markdown
Member

@glemaitre glemaitre Aug 11, 2023

Choose a reason for hiding this comment

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

It is true. Here, this is the typical XOR example. We should probably have this example for all classifiers but not specifically for the SVM.

@adrinjalali
Copy link
Copy Markdown
Member

Thanks for the PR, but we've decided to remove this example and it's done in #27108

@marenwestermann
Copy link
Copy Markdown
Member

I'm closing this PR. Thank you @TheCheshireCat6 for your work. If you like you could submit another PR that contributes to the same meta-issue as this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants