Skip to content

Translate ja event#241

Merged
lewtun merged 9 commits into
huggingface:mainfrom
trtd56:translate_ja_event
Jun 20, 2022
Merged

Translate ja event#241
lewtun merged 9 commits into
huggingface:mainfrom
trtd56:translate_ja_event

Conversation

@trtd56

@trtd56 trtd56 commented Jun 11, 2022

Copy link
Copy Markdown
Contributor

Hi @lewtun @hiromu166
In this PR, I translate event/1.mdx.

This is my first-time pull request. If there is some mistakes, please tell me.

#114

@HuggingFaceDocBuilderDev

HuggingFaceDocBuilderDev commented Jun 11, 2022

Copy link
Copy Markdown

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

@hiromu166

Copy link
Copy Markdown
Contributor

Hi, @trtd56.
Thank you for translating event/1.mdx. I'll review this PR soon.
By the way, could you set Reviewers in the upper right corner? You can request review here.

https://docs.github.com/ja/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/requesting-a-pull-request-review

@trtd56
trtd56 marked this pull request as draft June 14, 2022 20:33
@trtd56
trtd56 marked this pull request as ready for review June 14, 2022 20:33
@trtd56

trtd56 commented Jun 14, 2022

Copy link
Copy Markdown
Contributor Author

Hi, @hiromu166.

Sorry, I can't seem to assign a reviewer.
No gear mark for settings.

スクリーンショット 2022-06-15 054546

Is this note which you linked to relevant?

メモ: プルリクエスト作成者は、リポジトリ オーナー、またはリポジトリへの書き込みアクセス権を持つコラボレーターのどちらかでなければ、レビューをリクエストすることはできません。

@lewtun lewtun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you so much for this translation @trtd56 and welcome to the Hugging Face ecosystem! It's really cool that this is your first pull request 🤗

I left a few minor comments about the translation, but otherwise this is looking really good to me!

Comment thread chapters/ja/_toctree.yml Outdated
- title: Hugging Faceコースのイベント
sections:
- local: event/1
title: Chapter2公開記念イベント No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's been some time since I studied Japanese, but in the English version we refer to this as "Part 2 Release Event", so should the translation be something like

第2部発売記念イベント

I'm just asking why you chose to replace "Part 2" with "Chapter 2" :) Just FYI we use "Part 2" to refer to chapters 4-8

@younesbelkada younesbelkada Jun 17, 2022

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I second @lewtun's comments , to make it consistent with the other PR (#244) I think you should put パート2 and not Chapter2

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the confirmation.

「パート2公開記念イベント」
is a good idea?

I had thought Part2 mean is this repository's chapter2.
However, I should put PART2 to make it consistent with PR (#244) as @younesbelkada said.

And, I think "Release" should be translated to "公開" because I predict this "Release" means "publish".
"発売" means "sale".

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Perfect thank you very much! Agreed with you

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"パート2公開記念イベント" sounds good to me too!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I fixed in 6075579

Thanks!

Comment thread chapters/ja/event/1.mdx Outdated
@@ -0,0 +1,204 @@
# Chapter2公開記念イベント

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Similar comment as above

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I fixed in 6075579

Thanks!

Comment thread chapters/ja/event/1.mdx Outdated
@@ -0,0 +1,204 @@
# Chapter2公開記念イベント

Chapter 2のコース公開後、Chapter 3のFine-tuningのコースの前に、2日間のトークライブイベントが開催されました。

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe we can use "Part 2" and "fine-tuning workshop" instead of "Chapter 2" and "Chapter 3"?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You're right.

The reason I translated it this way is that I thought "a fine-tuning sprint" would not correspond after part 2. (It should be part 3 after part 2).

But I thought I should be faithful to the original text, so I will translate it as:
「パート2のコース公開後、Fine-tuningのスプリントの前に、2日間のトークライブイベントが開催されました。」

What do you think?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes agreed with @trtd56 ! ;)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, your new proposal looks great to me!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I fixed it in a5887ee

@younesbelkada younesbelkada mentioned this pull request Jun 17, 2022
7 tasks

@hiromu166 hiromu166 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for translating!
This looks almost perfect for me.

Comment thread chapters/ja/event/1.mdx Outdated
彼女の研究は、アッシュ・カーター国防長官や米国盲人財団から表彰され、複数のテクノロジー企業で導入されています。
ちなみに彼女はガーデニングと犬、猫が好きです。

**Matthew Watson and Chen Qian:** *Keraによる自然言語処理のワークフロー*

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Keraによる -> Kerasによる

@trtd56 trtd56 Jun 19, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I fixed in c4d530b

Thanks!

@HiromuHota HiromuHota mentioned this pull request Jun 19, 2022
7 tasks
@trtd56

trtd56 commented Jun 20, 2022

Copy link
Copy Markdown
Contributor Author

@lewtun @hiromu166 @younesbelkada @HiromuHota

I corrected the points that were pointed out.
Please review again.

@younesbelkada

Copy link
Copy Markdown
Contributor

Hi @trtd56 ! Looks very good to me :) thanks for the corrections

@lewtun lewtun left a comment

Copy link