-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
DOC: Link Examples for SVR, NuSVR, and SVM User Guide #30201
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
DOC: Link Examples for SVR, NuSVR, and SVM User Guide #30201
Conversation
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.
@virchan Could you please review and verify the recent push? I've linked the relevant examples for |
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.
Thank you for updating the PR @lunovian! I have a few additional suggestions.
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.
Remove redundant lines and symbols, rewrite words in sklearn/svm/_classes.py
Somehow, the error appears in the Azure Pipeline, which I didn't touch or anything related to them. Is this just a bug, or it was my fault? |
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.
So far, so good! I have a few more suggestions.
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.
LGTM!
C.c. @adrinjalali.
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.
LGTM. Thanks @lunovian
Thank you @virchan and @OmarManzoor for supporting me with this. As a newbie, I truly appreciate the opportunity to learn new things. |
Reference Issues/PRs
Towards #26927
What does this implement/fix? Explain your changes.
This PR adds links to relevant examples in the docstrings for
SVR
andNuSVR
and updates the Support Vector Machines user guide to enhance discoverability and provide practical usage guidance for these estimators. The following links were added:SVR
andNuSVR
: Linked toplot_svm_regression.py
exampleRelated Example Files:
plot_model_complexity_influence.py
Any other comments?
These additions improve documentation clarity, making it easier for users to find relevant examples directly from the API documentation. Let me know if any additional adjustments are required!