DOC Add Links to plot_lof_novelty_detection example#31405
DOC Add Links to plot_lof_novelty_detection example#31405adrinjalali merged 3 commits intoscikit-learn:mainfrom
Conversation
StefanieSenger
left a comment
There was a problem hiding this comment.
Thank you, @SiyuJin-1, I think that's a good spot for the example.
A user would also get there if they clicked the image, but that's not very obvious.
I would suggest to make it all a bit shorter as in my suggestion.
Co-authored-by: Stefanie Senger <91849487+StefanieSenger@users.noreply.github.com>
Co-authored-by: Stefanie Senger <91849487+StefanieSenger@users.noreply.github.com>
|
Hi @StefanieSenger, thank you very much for your helpful feedback! |
StefanieSenger
left a comment
There was a problem hiding this comment.
Thank you @SiyuJin-1, that looks very good.
Would you have a look, @adrinjalali? or @betatim?
adrinjalali
left a comment
There was a problem hiding this comment.
For future reference, we usually tend to not add explicit links to examples where there's a link to their plot.
The issue is that it's not clear in the UI that the user can click on the plot to go to the example, and that's something we need to fix in general for the whole link to examples template. But since this PR is already here, I'm happy to merge.
Co-authored-by: Jsy <jsy@zhouhantaodeMacBook-Pro.local> Co-authored-by: Stefanie Senger <91849487+StefanieSenger@users.noreply.github.com>
Co-authored-by: Jsy <jsy@zhouhantaodeMacBook-Pro.local> Co-authored-by: Stefanie Senger <91849487+StefanieSenger@users.noreply.github.com>
Co-authored-by: Jsy <jsy@zhouhantaodeMacBook-Pro.local> Co-authored-by: Stefanie Senger <91849487+StefanieSenger@users.noreply.github.com>
Co-authored-by: Jsy <jsy@zhouhantaodeMacBook-Pro.local> Co-authored-by: Stefanie Senger <91849487+StefanieSenger@users.noreply.github.com>
Co-authored-by: Jsy <jsy@zhouhantaodeMacBook-Pro.local> Co-authored-by: Stefanie Senger <91849487+StefanieSenger@users.noreply.github.com>
Reference Issues/PRs
Towards #30621.
What does this implement/fix? Explain your changes.
Added a link to the plot_lof_novelty_detection example to outlier_detection.rst. The example was already referenced via a figure target, but adding a link makes it more discoverable.