Skip to content

Conversation

@manueldeprada
Copy link
Contributor

Removes Constrained Beam Search generation strategy from the codebase. Directs users to the transformers-community/constrained-beam-search repository.

It has been a warning for a few releases, but now trust_remote_code=True is required to run constrained beam search.

Depends on #40480 and #40495

@manueldeprada manueldeprada requested a review from gante August 28, 2025 09:47
@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

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

important: we need to deprecate classes before we remove them (the classes we remove from __init__ were not deprecated)

@manueldeprada
Copy link
Contributor Author

build-doc

@github-actions
Copy link
Contributor

@manueldeprada manueldeprada requested a review from gante August 29, 2025 08:39
Copy link
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 🧹 🧹 🧹

(added a few minor nits)

@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

[For maintainers] Suggested jobs to run (before merge)

run-slow: csm, dia, qwen2_5_omni, recurrent_gemma, rwkv, whisper

@manueldeprada manueldeprada enabled auto-merge (squash) September 1, 2025 12:33
@manueldeprada manueldeprada merged commit 8564e21 into huggingface:main Sep 1, 2025
24 checks passed
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