Skip to content

DOC Add link for prediction latency plot for classification benchmark#30462

Closed
kayo09 wants to merge 7 commits intoscikit-learn:mainfrom
kayo09:examples-26927
Closed

DOC Add link for prediction latency plot for classification benchmark#30462
kayo09 wants to merge 7 commits intoscikit-learn:mainfrom
kayo09:examples-26927

Conversation

@kayo09
Copy link
Copy Markdown

@kayo09 kayo09 commented Dec 11, 2024

Towards #26927

Add example link documentation for ploting benchmarks for forest covertype dataset

edit: thank you @StefanieSenger :)

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 11, 2024

✔️ Linting Passed

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

Generated for commit: d5af721. Link to the linter CI: here

@kayo09 kayo09 changed the title DOC Add link for prediction latency plot DOC Add link for prediction latency plot for classification benchmark Dec 11, 2024
@StefanieSenger
Copy link
Copy Markdown
Member

StefanieSenger commented Dec 16, 2024

Hi @kayo09,

I see you are trying to make your first contribution to scikit-learn in our meta issue for adding links to the documentation. Two things:

  1. Your suggestion unfortunately does not add the links to the docs build with sphinx. Please refer to these tipps to learn what this issue tries to archive and push again to this branch.

  2. Please do not write "Fix: 26927" in the PR description, because his leads to automatically closing our issue when your PR would be merged. Use something like "Towards 26927" instead.

@StefanieSenger
Copy link
Copy Markdown
Member

I will close this PR after being stalled for so long. I don't think the author will return.

@kayo09
Copy link
Copy Markdown
Author

kayo09 commented Jul 5, 2025

I will close this PR after being stalled for so long. I don't think the author will return.

Never say never

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.

2 participants