Skip to content

[FR] Continued proofreading and visual corrections#167

Closed
lbourdois wants to merge 21 commits into
huggingface:mainfrom
lbourdois:main
Closed

[FR] Continued proofreading and visual corrections#167
lbourdois wants to merge 21 commits into
huggingface:mainfrom
lbourdois:main

Conversation

@lbourdois

@lbourdois lbourdois commented May 7, 2022

Copy link
Copy Markdown
Contributor

Hi,

In this PR:

  • I made some visual corrections on chapters 0 to 4 in French. They mainly consist of fixing the problem of the website that does not taking Markdown tags into account in the titles (and after exploration also in the end-of-chapter quiz and between tags) as mentioned at the end of the PR [FR] Translation of chapter 2 & event + Review of chapters 0 & 5 #106 discussion. The fix is to replace the tags in Mardown with those used in HTML.
  • added a review of chapters 5, 6, 8 and event in French (in addition to the pure review, it also takes the review of form by adopting the visual corrections explained in the previous dash).
  • I made a small correction on the English file of the event chapter to say that the book on Transformers is now published + fix links in chapter 3
  • adds the translation of chapter 9

The only thing missing is the reviewing of chapters 7 (and maybe chapter 9 visually) and the translation of the videos for the French version to be finished :D

lbourdois added 5 commits May 7, 2022 17:10
- review chapter 5, 8 & event
- visual fix (italic in # lines) on chapter 0 to 4
the book is now available
fix tag
the comma 🙃
It was not the comma...
I don't know what the problem is. Probably HTML tags to generate a link. So I'll remove them, the user will just have to copy/paste the link in parenthesis
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

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

lbourdois added 10 commits May 7, 2022 18:37
fix italic
fix italic
fix italic
use better word for "check"
little fix
visual fix
visual fix
little fix
fix italic
code_formatter.py
@lbourdois

Copy link
Copy Markdown
Contributor Author

The HuggingFaceDocBuilderDev allowed me to view my corrections and realize that some italic tags were missing.
Everything is clean now 🥳

lbourdois added 6 commits May 14, 2022 16:36
review chapter 6
fix link
fix AlBERT
Add chapter 9
add chapter 9 to toctree

@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 for another epic PR @lbourdois 🤯 !!

There's a few merge conflicts to resolve from some PRs I just merged, but after they're resolved I think this is good to go 🚀

The `Trainer` will work out of the box on multiple GPUs or TPUs and provides lots of options, like mixed-precision training (use `fp16 = True` in your training arguments). We will go over everything it supports in Chapter 10.

This concludes the introduction to fine-tuning using the `Trainer` API. An example of doing this for most common NLP tasks will be given in [Chapter 7](course/chapter7), but for now let's look at how to do the same thing in pure PyTorch.
This concludes the introduction to fine-tuning using the `Trainer` API. An example of doing this for most common NLP tasks will be given in [Chapter 7](/course/chapter7), but for now let's look at how to do the same thing in pure PyTorch.

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.

Nice catch!

La plupart du cours repose sur le fait que vous ayez un compte Hugging Face. Si vous n'en disposez pas d'un, nous vous recommandons d'en créer un dès maintenant : [créer un compte](https://huggingface.co/join).
## Utilisation d'un <i>notebook</i> Google Colab

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.

Ah that's a cool hack :)

classifier(
"This is a course about the Transformers library", # C'est un cours sur la bibliothèque Transformers
"This is a course about the Transformers library",
# C'est un cours sur la bibliothèque Transformers

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.

This is a really cool example to show readers what the text is in their native tongue - nice idea!

Since French has reasonable coverage model-wise, one idea for the future would be to change the code so that we use French models instead of English ones. This will require some work on our side to make sure the Colab notebooks are generated per language, but could be interesting to explore in the future

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.

This is something I had in mind. I'll submit a list of ideas to you on Discord.

@lbourdois lbourdois mentioned this pull request May 16, 2022
This pull request was closed.
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.

3 participants