Skip to content

Fix model integration ci#26322

Merged
ArthurZucker merged 32 commits intohuggingface:mainfrom
ArthurZucker:fix-model-integration-ci
Oct 2, 2023
Merged

Fix model integration ci#26322
ArthurZucker merged 32 commits intohuggingface:mainfrom
ArthurZucker:fix-model-integration-ci

Conversation

@ArthurZucker
Copy link
Collaborator

What does this PR do?

Fixes the CI broken by #23909

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Sep 21, 2023

The documentation is not available anymore as the PR was closed or merged.

@ArthurZucker ArthurZucker marked this pull request as ready for review September 28, 2023 10:09
@LysandreJik LysandreJik self-assigned this Oct 2, 2023
Copy link
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for investigating and fixing the failing tests @ArthurZucker

@ArthurZucker ArthurZucker merged commit 63864e0 into huggingface:main Oct 2, 2023
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.

length of tokenizer changes when using main branch causing batch_decode to fail

3 participants