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
Translated tutorials into French Part 1 (OpenMind #2782 and #2779) #2881
Conversation
Check out this pull request on You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey,
Thanks for this PR!
Did you use a software to translate to French?
I have the feeling that some of the expressions sound a bit awkward.
If you did so, then you might want to spend some time proof reading the translation.
examples/tutorials/translations/french/Part 01 - The Basic Tools of Private Deep Learning.ipynb
Outdated
Show resolved
Hide resolved
examples/tutorials/translations/french/Part 01 - The Basic Tools of Private Deep Learning.ipynb
Outdated
Show resolved
Hide resolved
examples/tutorials/translations/french/Part 01 - The Basic Tools of Private Deep Learning.ipynb
Outdated
Show resolved
Hide resolved
"hotkey": "alt-t", | ||
"langInMainMenu": true, | ||
"sourceLang": "en", | ||
"targetLang": "fr", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably don't want this translation metadata in the notebooks
"\n", | ||
"- [Star PySyft] (https://github.com/OpenMined/PySyft)\n", | ||
"\n", | ||
"### Rejoignez notre Slack!\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would se joindre
be the correct verb here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes @TTitcombe you're right!
I would love to help on this PR and translate the remaining notebooks. I am currently diving back to the project and it can be a good way to catch up. |
…ls of Private Deep Learning.ipynb Co-Authored-By: Tom <32938439+TTitcombe@users.noreply.github.com>
Thank you for catching those. Co-Authored-By: Tom <32938439+TTitcombe@users.noreply.github.com>
I don't think you can push here maybe you'll want ot open a dedicated PR :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@LaRiffle One test under test coverage has been cancelled. Is it okay to merge without the test passing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving and merging French translations for part 1
Issue resolved #2881
@Metrix1010 Fix those requested changes. Once fixed, I will be able to merge them. |
Please let me know if this is correct. I used git rebase to update my local fork but I did not use the force command. I used merge, pull and push. |
This pull request has been marked stale because it has been open 30 days with no activity. Leave a comment or remove the |
examples/tutorials/translations/french/Part 01 - The Basic Tools of Private Deep Learning.ipynb