-
Notifications
You must be signed in to change notification settings - Fork 374
Template for providers logos in dark and light mode + images in HF dataset #1685
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
Wauplin
commented
Apr 9, 2025
- logos are now stored under https://huggingface.co/datasets/huggingface/documentation-images/tree/main/inference-providers/logos
- logos and badges and both in light and dark modes
- use templates to avoid maintaining urls and stuff manually (1 template for logos, 1 template for badges)
- add instruction in comments
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
| </div> | ||
|
|
||
| <div class="flex justify-center"> | ||
| <a href="https://huggingface.co/sambanova" target="_blank"> |
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.
| <a href="https://huggingface.co/sambanova" target="_blank"> | |
| <a href="https://huggingface.co/sambanovasystems" target="_blank"> |
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.
no s at the end i think
| @@ -0,0 +1,6 @@ | |||
| <div class="flex justify-center"> | |||
| <a href="https://huggingface.co/{{provider}}" target="_blank"> | |||
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.
actually... the provider org pages are subtly ≠ from the provider id (there are two exceptions i think)
so i think you need to maintain another mapping, or just call the template with an explicit argument
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.
as @hanouticelina caught as well :)
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.
ah yes it was auto-generated from the provider names, yes let's add a provider hf page mapping then
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.
pre-approving :)
| @@ -0,0 +1,6 @@ | |||
| <div class="flex justify-center"> | |||
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.
| <div class="flex justify-center"> | |
| <div class="flex"> |
i think it looks better left-aligned no?
|
Thanks for the comments 😬 I fixed the org handles with a new mapping (to maintain...) and removed the |
|
tested all "follow us" buttons under https://moon-ci-docs.huggingface.co/docs/inference-providers/pr_1685/en/providers/cerebras and they work fine. Merging |