-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
DOC added links for plot_affinity_propagation.py #29759
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
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.
Just a small comment. Otherwise LGTM.
For an example, see :ref:`examples/cluster/plot_affinity_propagation.py | ||
<sphx_glr_auto_examples_cluster_plot_affinity_propagation.py>`. | ||
For an example usage, | ||
see :ref:`sphx_glr_auto_examples_cluster_plot_affinity_propagation.py`. |
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.
I would also mention this example here: https://scikit-learn.org/stable/auto_examples/applications/plot_stock_market.html#clustering-using-affinity-propagation
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.
I have added the example, and rebased the branch.
Can you pull the changes from upstream main into your branch? It should fix the CI failures. |
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
ping @adrinjalali
@ivanpan0626 in the future please make sure you create a feature branch locally instead of working on your |
Reference Issues/PRs
issue #26927
What does this implement/fix? Explain your changes.
Added example links for plot_affinity_propagation.py
Added example links for plot_dbscan.py
Any other comments?