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

Update NLLB documentation to reflect current behavior #1331

Open
sotwi opened this issue Jul 11, 2024 · 1 comment
Open

Update NLLB documentation to reflect current behavior #1331

sotwi opened this issue Jul 11, 2024 · 1 comment

Comments

@sotwi
Copy link

sotwi commented Jul 11, 2024

Doc request
Substitute mentions of tokenizer.lang_code_to_id from NLLB documentation.

Additional context
huggingface/transformers#29834 removed lang_code_to_id from NLLBTokenizer and NLLBTokenizerFast but the documentation on the website still makes use of it:

https://github.com/huggingface/transformers/blob/52585019a17f6033df64e6ad4222d22a1f993c61/docs/source/en/model_doc/nllb.md?plain=1#L104

https://github.com/huggingface/transformers/blob/52585019a17f6033df64e6ad4222d22a1f993c61/docs/source/en/model_doc/nllb.md?plain=1#L129

https://github.com/huggingface/transformers/blob/52585019a17f6033df64e6ad4222d22a1f993c61/docs/source/en/model_doc/nllb.md?plain=1#L178

@julien-c
Copy link
Member

Hi, can you open this issue on the transformers repo instead?

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

No branches or pull requests

2 participants