Hugging Quants org
No description provided.
Hugging Quants org

Very nice, looks great to me!

Question about the "instead of" comments in the final snippet. If the OpenAI API works against the locally deployed model, maybe we could add another snippet to show how to use it?

Hugging Quants org

Thanks for the review @pcuenq ! Maybe we should then add a collapsable with the openai snippet to avoid having a huge model card? Or just add it afterwards? Note that I've just included vLLM too, and that may be already way too much for the card, WDYT? I'm all ears on what's the best 🤗

Hugging Quants org

I think it's great to show different options so users can apply the solution that works best for them. We can use a ToC at the beginning of the model card or collapsable sub-sections, as you prefer :)

Hugging Quants org

Just applied those changes, thanks for the review @pcuenq , feel free to merge whenever! Then I'll apply the updates to the rest of the supported models 🤗

pcuenq changed pull request status to merged

Sign up or log in to comment