Skip to content

Conversation

mogith-pn
Copy link

@mogith-pn mogith-pn commented Oct 8, 2025

Hi Team,
Added clarifai as provider for inference.
Related PR - huggingface/huggingface.js#1781

@mogith-pn mogith-pn closed this Oct 8, 2025
@mogith-pn mogith-pn reopened this Oct 8, 2025
@mogith-pn mogith-pn changed the title added clarifai as provider Add clarifai as Inference provider Oct 8, 2025
Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

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

Thanks for opening this PR @mogith-pn! Implementation can be simplified, otherwise looks good :)

@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.

@Wauplin
Copy link
Contributor

Wauplin commented Oct 8, 2025

Could you also run make style locally and commit the changes please? It looks like you have some code quality issues (likely due to a "autoformat on save")

@mogith-pn
Copy link
Author

Thanks for the quick review @Wauplin . Addressed the comments.

@mogith-pn mogith-pn requested a review from Wauplin October 8, 2025 15:22
Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

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

Looks ready! 🤗

Will keep it open until clarifai is integrated as staging so we can test it "for real".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants