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

convert_tokens_to_string documentation bug #10303

Closed
cronoik opened this issue Feb 20, 2021 · 0 comments · Fixed by #10304
Closed

convert_tokens_to_string documentation bug #10303

cronoik opened this issue Feb 20, 2021 · 0 comments · Fixed by #10304

Comments

@cronoik
Copy link
Contributor

cronoik commented Feb 20, 2021

The documentation states that convert_tokens_to_string would convert a sequence of token ids in a single string.

That is actually not correct as it converts a sequence of tokens. The method that converts a sequence of token ids is the decode method.

Documentation: @sgugger

cronoik added a commit to cronoik/transformers that referenced this issue Feb 20, 2021
@cronoik cronoik mentioned this issue Feb 20, 2021
5 tasks
sgugger pushed a commit that referenced this issue Feb 20, 2021
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 a pull request may close this issue.

1 participant