Skip to content
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

Add Fireworks AI provider + instructions for new provider #2848

Merged
merged 45 commits into from
Feb 12, 2025

Conversation

Wauplin
Copy link
Contributor

@Wauplin Wauplin commented Feb 11, 2025

Requires #2836 to be merged first.

This PR adds support for the https://fireworks.ai/ provider for conversational task.
I have also added a short doc file to explain how to add new providers in the future (instructions still to be refined but good draft for a v1 IMO). Adding a new provider to the Python client assumes the provider is already integrated on the Hub + huggingface.js client. This simplifies a lot the process.

@HuggingFaceDocBuilderDev

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.

Copy link
Contributor

@hanouticelina hanouticelina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! could you update the table of supported providers here as well ?

@hanouticelina hanouticelina added the inference Anything related to InferenceClient, providers, etc. label Feb 11, 2025
@Wauplin
Copy link
Contributor Author

Wauplin commented Feb 11, 2025

could you update the table of supported providers here as well ?

Done in 4324dea

Base automatically changed from add-dynamic-inference-provider-mapping to main February 11, 2025 17:10
@Wauplin
Copy link
Contributor Author

Wauplin commented Feb 11, 2025

@hanouticelina #2836 is merged so this one is now ready for review :)

Copy link
Contributor

@hanouticelina hanouticelina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you @Wauplin! Glad to see that adding a new provider is easy and straightforward 😄

@Wauplin Wauplin merged commit a365b7d into main Feb 12, 2025
17 checks passed
@Wauplin Wauplin deleted the add-fireworks-provider branch February 12, 2025 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inference Anything related to InferenceClient, providers, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants