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

[i18n-ar] Translated file : docs/source/ar/fast_tokenizers.md into Arabic #33034

Merged

Conversation

AhmedAlmaghz
Copy link
Contributor

What does this PR do?

Translated the docs/source/ar/fast_tokenizers.md file of the documentation to Arabic العربية. I use transformers for my side-project and wanted to contribute. As a beginner, this seemed appropriate.

Some questions:

  • I can't access the pr documentation endpoint for Arabic العربية. Is there a missing step?
  • I left all the other files alone after copying the English docs. Is this acceptable for future updates or should I add contents to the _toctree.yml only when translation is complete? Should I delete all other files for a cleaner review?
  • In Arabic, people use both Transformer and المحولات to describe the model. However, as this is more of a product of HuggingFace I opted not to translate it. May you please let me know which you would prefer?

Thank you in advance for your review.

Part of #32435

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline, Pull Request section?
  • Was this discussed/approved via a Github issue or the forum ?Please add a link to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the documentation guidelines, and here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

@stevhliu , @MKhalusova , @abodacs, may you please review this PR?

@AhmedAlmaghz
Copy link
Contributor Author

AhmedAlmaghz commented Oct 10, 2024

من هنا نبداء القسم الجديد
@abodacs

DEVELOPER GUIDES

Copy link
Contributor

@abodacs abodacs left a comment

Choose a reason for hiding this comment

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

سﻻم عليكم
@AhmedAlmaghz

تمت المراجعة جزاء الله خيرا

docs/source/ar/_toctree.yml Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
docs/source/ar/fast_tokenizers.md Outdated Show resolved Hide resolved
@AhmedAlmaghz AhmedAlmaghz marked this pull request as ready for review October 26, 2024 20:54
@AhmedAlmaghz
Copy link
Contributor Author

Thank you @abodacs

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

@stevhliu stevhliu left a comment

Choose a reason for hiding this comment

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

Thanks for the translation and thanks @abodacs for the review! 👏

@stevhliu stevhliu merged commit a17f287 into huggingface:main Oct 28, 2024
9 checks passed
@AhmedAlmaghz AhmedAlmaghz deleted the Add_docs_source_ar_fast_tokenizers.md branch October 30, 2024 03:22
BernardZach pushed a commit to BernardZach/transformers that referenced this pull request Dec 5, 2024
…Arabic (huggingface#33034)

* Add docs/source/ar/fast_tokenizers.md to Add_docs_source_ar_fast_tokenizers.md

* Update _toctree.yml

* Update _toctree.yml

* Update docs/source/ar/_toctree.yml

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

---------

Co-authored-by: Abdullah Mohammed <[email protected]>
BernardZach pushed a commit to innovationcore/transformers that referenced this pull request Dec 6, 2024
…Arabic (huggingface#33034)

* Add docs/source/ar/fast_tokenizers.md to Add_docs_source_ar_fast_tokenizers.md

* Update _toctree.yml

* Update _toctree.yml

* Update docs/source/ar/_toctree.yml

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

* Update docs/source/ar/fast_tokenizers.md

Co-authored-by: Abdullah Mohammed <[email protected]>

---------

Co-authored-by: Abdullah Mohammed <[email protected]>
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.

4 participants