-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[docs] Supported models tables #364
Conversation
The documentation is not available anymore as the PR was closed or merged. |
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.
Very nice! I think placing it on the index immediately gives users an idea of whether their use case is supported or not 👍
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
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.
Thank you @MKhalusova for updating the doc page with the supported models tables, LGTM! 🚀
Left a suggestion based on recently merged PRs
|
||
Supported methods include: | ||
If you are new to PEFT, get started by reading the [Quicktour](quicktour) guide and conceptual guides for [LoRA](/conceptual_guides/lora) and [Prompting](/conceptual_guides/prompting) methods. |
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.
🚀
Co-authored-by: Sourab Mangrulkar <13534540+pacman100@users.noreply.github.com>
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. |
As a suggestion, this PR moves the tables listing supported models to the index page since the supported methods are also listed there.
In addition, this PR adds links to the quicktour and conceptual guides to help beginners navigate the docs.