-
Notifications
You must be signed in to change notification settings - Fork 487
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
Rework doc homepage #1624
Rework doc homepage #1624
Conversation
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. |
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.
Looks good, thanks @regisss !
<a class="!no-underline border dark:border-gray-700 p-5 rounded-lg shadow hover:shadow-lg" href="./torch_fx/overview" | ||
><div class="w-full text-center bg-gradient-to-br from-green-400 to-green-500 rounded-lg py-1.5 font-semibold mb-5 text-white text-lg leading-relaxed">Torch FX</div> | ||
<p class="text-gray-700">Create and compose custom graph transformations to optimize PyTorch Transformers models with <span class="underline" onclick="event.preventDefault(); window.open('https://pytorch.org/docs/stable/fx.html#', '_blank');">Torch FX</span></p> | ||
</a> | ||
</div> | ||
</div> |
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.
Not related to this PR but the whole color layout is burning my eyes, will open a new PR to propose something (not that it'll be any better lol)
</div> | ||
|
||
|
||
## Open-source integrations |
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.
Not sure about "Open-source" as it's also true for the previous section, but would go with something like "hardware-agnostic"
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 meant that for instance Furiosa's or Habana's SDKs are not open-source but they are integrated into Optimum, unlike ORT, FX etc.
The idea I would like to convey here is that HPPs are not the only thing we do and that we also work on the integration of other popular libraries into Optimum. Maybe there is a better way to say it?
What does this PR do?
This PR introduces a new layout for the doc homepage. It splits subpackages into two sections (hardware partners and open-source integrations). This gives a clearer view of what the various subpackages provide.
Before submitting