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

[Bot] Update inference types #2791

Merged
merged 4 commits into from
Feb 4, 2025
Merged

Conversation

HuggingFaceInfra
Copy link
Contributor

This PR updates the inference types. It has been generated by running:

# Generate Python code
pnpm run inference-codegen  # from @huggingface/tasks-gen

# Copy to this repository
cp -r ./huggingface.js/packages/tasks/.python_generated/* ./huggingface_hub/src/huggingface_hub/inference/_generated/types

# Clean things
make inference_update
make style

This PR was automatically created by the Update Inference Types workflow.

Make sure the changes are correct before merging.

@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 Jan 28, 2025

Ping @Rocketknight1 this is the follow-up PR after #2712
Re: generation_parameters instead of generation_kwargs

@Wauplin
Copy link
Contributor

Wauplin commented Jan 28, 2025

follow-up PR: #2791

@HuggingFaceInfra HuggingFaceInfra force-pushed the update-inference-types-automated-pr branch 6 times, most recently from 121459b to 4662b7b Compare February 4, 2025 10:21
@HuggingFaceInfra HuggingFaceInfra force-pushed the update-inference-types-automated-pr branch from 4662b7b to e8ed5f7 Compare February 4, 2025 10:25
@Wauplin
Copy link
Contributor

Wauplin commented Feb 4, 2025

Merging it right now without the generate_kwargs update.

@Wauplin Wauplin merged commit f2e1889 into main Feb 4, 2025
17 checks passed
@Wauplin Wauplin deleted the update-inference-types-automated-pr branch February 4, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants