Skip to content

Add Japanese trasnlation of chapter 1 / 2 to 3#363

Merged
younesbelkada merged 6 commits into
huggingface:mainfrom
blackdoor571:main
Nov 10, 2022
Merged

Add Japanese trasnlation of chapter 1 / 2 to 3#363
younesbelkada merged 6 commits into
huggingface:mainfrom
blackdoor571:main

Conversation

@blackdoor571

Copy link
Copy Markdown
Contributor

No description provided.

@ArthurZucker

Copy link
Copy Markdown

Awesome work! 🤗
In order to have a preview, you need to add the chapters to the chapters/ja/_toctree.yml as in #363 👍🏻
Then we'll be able to make sure it renders well

@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Comment thread chapters/ja/chapter1/3.mdx
Comment thread chapters/ja/chapter1/3.mdx
Comment thread chapters/ja/chapter1/3.mdx
Comment thread chapters/ja/chapter1/3.mdx
@fxmarty

fxmarty commented Nov 4, 2022

Copy link
Copy Markdown
Contributor

This is super cool ️‍🔥 I was thinking it could be nice to have examples in Japanese throughout the course, what do you think? Just a suggestion!

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

Thanks a lot for this great translation @blackdoor571 and welcome to the 🤗 community!

I think @fxmarty's idea to use Japanese models is nice, but probably best done directly in the notebooks to ensure the course content is 1-1 across languages. No need to do that yourself, but just making a note here for other translators :)

There's a small code styling issue with the PR - could you please install the requirements and run make style?

Comment thread chapters/ja/chapter1/3.mdx
@fxmarty

fxmarty commented Nov 4, 2022

Copy link
Copy Markdown
Contributor

@lewtun Makes sense. Here's an example of how it was done in the French translation, with comments: https://huggingface.co/course/fr/chapter1/3?fw=pt . Honestly I'm not fan of putting comments everywhere (everything feels like less readable), but I see the point where there may be readers not able to grasp examples in English. I agree we could keep everything in English and leave translated example in future notebooks though.

@lewtun

lewtun commented Nov 10, 2022

Copy link
Copy Markdown
Member

Hey @blackdoor571 thanks for this great translation! We just merged #359 which created some merge conflicts on the _toctree.yml file. Would you mind rebasing / merging with main and pushing again? Thanks!

@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@younesbelkada

Copy link
Copy Markdown
Contributor

Thank you very much @blackdoor571 for your great contribution!! Merging

@younesbelkada
younesbelkada merged commit 7081ab6 into huggingface:main Nov 10, 2022
GITbaburin pushed a commit to GITbaburin/course that referenced this pull request May 17, 2026
Add Japanese trasnlation of chapter 1 / 2 to 3
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.

6 participants