transformers icon indicating copy to clipboard operation
transformers copied to clipboard

[i18n-ar] Add File : `docs/source/ar/_toctree.yml`

Open AhmedAlmaghz opened this issue 1 year ago • 7 comments

What does this PR do?

Translate docs into Arabic issue-number #32435 @stevhliu , @MKhalusova , @abodacs This pull request contains changes for file : docs/source/ar/_toctree.yml

AhmedAlmaghz avatar Aug 14 '24 19:08 AhmedAlmaghz

Thanks

I have exams this week, and I'll do the review after. I'll start with this.

@AhmedAlmaghz

abodacs avatar Aug 16 '24 14:08 abodacs

@stevhliu, Please, review this PR.

AhmedAlmaghz avatar Aug 22 '24 15:08 AhmedAlmaghz

@abodacs اشكرك جداً على الملاحظات القيمة أنا سعيد بملاحظاتك وأتعلم منها أنت والآخرين أرجوا أن منك أن تسرع بالمراجعة لأن لدينا عمل كثير ونريد أن يستفيذ الجميع من الترجمه اشكر جهودك مرة أخرى إحترامي وتقديري

AhmedAlmaghz avatar Aug 26 '24 15:08 AhmedAlmaghz

@AhmedAlmaghz شكرا لمجهودك

ها

abodacs avatar Aug 26 '24 15:08 abodacs

LGTM cc: @stevhliu

abodacs avatar Aug 26 '24 15:08 abodacs

لضمان تنظيم العمل بيننا بشكل أفضل، أقترح سير العمل التالي:

  1. @abodacs: بمجرد أن تقوم بكتابة التعليق "إرسال الطلب للدمج"، مثل هذا.
  2. @AhmedAlmaghz: سأقوم بتحويل حالة طلب السحب من "مسودة" إلى "قيد المراجعة" لتتمكن من مراجعته وإرساله للدمج وفق الترتيب الذي تم تحديده في القضية #32435.
  3. @abodacs: تقوم بمراجعة الملفات المحددة في طلب السحب وتزويدنا بملاحظاتك واقتراحاتك على الترجمة. إذا كان كل شيء على ما يرام، ارجع إلى الخطوة 1.
  4. @AhmedAlmaghz: سأقوم بتنفيذ ملاحظاتك وأكتب تعليقاً: "تم التنفيذ".
  5. @abodacs: ارجع إلى الخطوة 3.
  6. @AhmedAlmaghz: ارجع إلى الخطوة 2.

ملاحظات:

  • هل تفضل أن يكون التواصل باللغة الإنجليزية أم العربية؟
  • لقد قمت بفتح عدة طلبات سحب وجعلتها في حالة مسودة، لأن تكرار نفس العمل يومياً غير ممكن بالنسبة لي.
  • أتمنى أن يكون هذا الإجراء مفيداً وسهلاً للتنفيذ.

تحياتي وتقديري.

################################################################

To better organize our work, I suggest the following workflow:

  1. @abodacs: As soon as you write the comment "Send request for merging," like this.
  2. @AhmedAlmaghz: I will change the status of the pull request from "Draft" to "Review" so you can review it and send it for merging according to the order set in the issue #32435.
  3. @abodacs: You will review the files specified in the pull request and provide us with your comments and suggestions on the translation. If everything is fine, go back to step 1.
  4. @AhmedAlmaghz: I will implement your feedback and comment: "Done."
  5. @abodacs: Go back to step 3.
  6. @AhmedAlmaghz: Go back to step 2.

Notes:

  • Do you prefer to communicate in English or Arabic?
  • I’ve opened several pull requests and left them as drafts because I can't repeat the same work every day.
  • I hope this process is useful and easy to follow.

Best regards.

AhmedAlmaghz avatar Aug 26 '24 18:08 AhmedAlmaghz

سﻻم عليكم @AhmedAlmaghz للاسف لم افهم الworkflow وحاولت ارسمه هذا ما وجده


graph TD
    A[Abodacs writes 'Send request for merging'] --> B{AhmedAlmaghz changes PR status to 'Review'}
    B --> C{Abodacs reviews and provides feedback}
    C --> D{AhmedAlmaghz implements feedback and comments 'Done'}
    D --> C
    C -- Everything is fine --> A
    D -- More changes needed --> B 

اقترح ان ترتيب المراجعه يكون على الشكل التالى

1- المراجع يبدأ من أعلى إلى اسفل من #32435 2- استثناء لذلك هما (إلى الان) #32696 و #33038

abodacs avatar Aug 27 '24 19:08 abodacs

Hi, thanks for fixing! All those other files aren't translated/exist yet so thats why the test is failing right now. Can you translate index.md to Arabic in this PR and then remove everything else? So it would look like:

- title: Get started
  sections:
  - local: index
    title: 🤗 المحولات

Once this PR is merged, then we can add #33038 😄

stevhliu avatar Aug 28 '24 16:08 stevhliu

Hi! Reminder that this comment needs to be resolved first before we can start merging the other PRs 🙂

stevhliu avatar Sep 04 '24 16:09 stevhliu

What is the solution? What do we do? @stevhliu

AhmedAlmaghz avatar Sep 04 '24 16:09 AhmedAlmaghz

  1. You'll have to remove everything in the toctree since none of those docs exist yet.
  2. Copy your translated index.md to this PR.

Your toctree should look like this:

- title: Get started
  sections:
  - local: index
    title: 🤗 المحولات

stevhliu avatar Sep 04 '24 16:09 stevhliu

Is that good? @stevhliu @abodacs

AhmedAlmaghz avatar Sep 04 '24 17:09 AhmedAlmaghz

@stevhliu Merge this PR, Please.

AhmedAlmaghz avatar Sep 08 '24 17:09 AhmedAlmaghz

Thank you @stevhliu After Merge, I will close all PRs,

AhmedAlmaghz avatar Sep 09 '24 17:09 AhmedAlmaghz

Done @stevhliu

AhmedAlmaghz avatar Sep 09 '24 17:09 AhmedAlmaghz

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.

Please, Mereg this PR. @stevhliu

AhmedAlmaghz avatar Sep 15 '24 19:09 AhmedAlmaghz

Thank you : @stevhliu @abodacs That's great work!

AhmedAlmaghz avatar Sep 16 '24 18:09 AhmedAlmaghz

Great work 🤗

Can you point @stevhliu @AhmedAlmaghz where I can add support in CSS for right to left?

screenshot-huggingface_co-2024_09_16-23_45_06

Arabic must start from the right in Docs sections like screenshot-huggingface_co-2024_09_16-23_52_00

abodacs avatar Sep 16 '24 20:09 abodacs

شكراً أخي عبدااله @abodacs على جهودك العظيمة اعتقد أن تعديل css يكون في القالب الخاص بتوثيق المحولات

AhmedAlmaghz avatar Sep 17 '24 00:09 AhmedAlmaghz

@stevhliu I opened PR huggingface/doc-builder#525

Could you check it?

abodacs avatar Sep 19 '24 20:09 abodacs