Skip to content

Remove deprecated batch_size parameter#37007

Merged
gante merged 3 commits intohuggingface:mainfrom
cyyever:batch_size
Mar 27, 2025
Merged

Remove deprecated batch_size parameter#37007
gante merged 3 commits intohuggingface:mainfrom
cyyever:batch_size

Conversation

@cyyever
Copy link
Copy Markdown
Contributor

@cyyever cyyever commented Mar 26, 2025

It's declared to be removed in 4.49.

@github-actions github-actions Bot marked this pull request as draft March 26, 2025 15:30
@github-actions
Copy link
Copy Markdown
Contributor

Hi 👋, thank you for opening this pull request! The pull request is converted to draft by default. The CI will be paused while the PR is in draft mode. When it is ready for review, please click the Ready for review button (at the bottom of the PR page). This will assign reviewers and trigger CI.

@cyyever cyyever marked this pull request as ready for review March 26, 2025 15:54
@Rocketknight1
Copy link
Copy Markdown
Member

cc @gante

Copy link
Copy Markdown
Contributor

@gante gante left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for fixing it! 💛

Added a missing step to complete this PR

Comment thread src/transformers/cache_utils.py
@cyyever cyyever force-pushed the batch_size branch 2 times, most recently from 1738711 to 496efdf Compare March 27, 2025 00:06
@cyyever cyyever requested a review from gante March 27, 2025 00:16
@cyyever cyyever changed the title Remove deprecated batch_size argument Remove deprecated batch_size parameter Mar 27, 2025
Copy link
Copy Markdown
Contributor

@gante gante 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 for iterating and making transformers better 🤗

@gante gante merged commit 6cc9c8d into huggingface:main Mar 27, 2025
18 checks passed
@cyyever cyyever deleted the batch_size branch March 28, 2025 02:18
zucchini-nlp pushed a commit to zucchini-nlp/transformers that referenced this pull request May 14, 2025
soghomon-b pushed a commit to soghomon-b/transformers that referenced this pull request Aug 24, 2025
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