8000 Link airspeed benchmarks page in the scikit-learn webpage · Issue #21898 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

Link airspeed benchmarks page in the scikit-learn webpage #21898

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.

Sign up for GitHub

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

Jump to bottom
Closed
lorentzenchr opened this issue Dec 6, 2021 · 5 comments · Fixed by #21956 or #22075
Closed

Link airspeed benchmarks page in the scikit-learn webpage #21898

lorentzenchr opened this issue Dec 6, 2021 · 5 comments · Fixed by #21956 or #22075
Assignees
@ohadmich
Labels
Documentation Easy Well-defined and straightforward way to resolve good first issue Easy with clear instructions to resolve

Comments

@lorentzenchr
Copy link
Member
lorentzenchr commented Dec 6, 2021 •
edited
Loading

Describe the issue linked to the documentation

Link the site https://scikit-learn.org/scikit-learn-benchmarks/ and https://github.com/scikit-learn/scikit-learn-benchmarks in the homepage https://scikit-learn.org.

Follow-up on #17026
@ogrisel
Copy link
Member
ogrisel commented Dec 9, 2021 •
edited
Loading

We could also find a way to generate a new (static) badge for the README.rst.

[![asv](http://img.shields.io/badge/benchmarked%20by-asv-blue.svg?style=flat)](https://scikit-learn.org/scikit-learn-benchmarks/ )

that should render as: asv

EDIT: actually the syntax above is for markdown. It would need to be adapted to use the reStructuredText syntax instead for our README.rst file.

@ogrisel ogrisel added Easy Well-defined and straightforward way to resolve help wanted good first issue Easy with clear instructions to resolve labels Dec 9, 2021
@ojeda-e
Copy link
Contributor
ojeda-e commented Dec 11, 2021

Can I help with this? @ogrisel

@ojeda-e ojeda-e mentioned this issue Dec 11, 2021
Merged
1 task
@lorentzenchr lorentzenchr reopened this Dec 13, 2021
@lorentzenchr
Copy link
Member Author

#21956 put the link in form of a badge to the README in github.

Question: Shall we also put the link somewhere on the website?

@ogrisel
Copy link
Member
ogrisel commented Dec 13, 2021

Question: Shall we also put the link somewhere on the website?

+1 for adding a link to doc/developers/contributing.rst in the intro paragraph of the section named Monitoring Performance.

@ohadmich
Copy link
Contributor

take

@ohadmich ohadmich mentioned this issue Dec 26, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ohadmich ohadmich

Labels
Documentation Easy Well-defined and straightforward way to resolve good first issue Easy with clear instructions to resolve
Projects
None yet
Milestone
No milestone
4 participants
@ogrisel @lorentzenchr @ojeda-e @ohadmich
0