Skip to content
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

[vi] cs-229-deep-learning #159

Merged
merged 12 commits into from
Apr 12, 2020

Conversation

tuananhhedspibk
Copy link
Contributor

No description provided.

@tuananhhedspibk tuananhhedspibk changed the title [vi] CheatSheet Deep Learning [vi] Deep Learning cheatsheet Jun 6, 2019
@tuananhhedspibk
Copy link
Contributor Author

@shervinea I want to translate this Deep Learning Cheatsheet into Vietnamese, could you add in progress label for me? Thank you a lot.

@shervinea
Copy link
Owner

@tuananhhedspibk Awesome to see your PR, thank you so much for your help on this! Yes, of course.

@shervinea shervinea changed the title [vi] Deep Learning cheatsheet [vi] Deep Learning Jun 6, 2019
@shervinea shervinea added the in progress Work in progress label Jun 6, 2019
@tuananhhedspibk
Copy link
Contributor Author

I have finished this translating, can you change the label to reviewer wanted for me, thank you so much.

@shervinea
Copy link
Owner

That's great, thank you @tuananhhedspibk! Please feel free to invite anyone you might know who could perform the review.

@shervinea shervinea added reviewer wanted Looking for a reviewer and removed in progress Work in progress labels Jun 7, 2019
@tuananhhedspibk
Copy link
Contributor Author

OK, I will invite the reviewer as soon as possible.

Copy link

@minhntm minhntm left a comment

Choose a reason for hiding this comment

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

I I think the word like 'batch', 'gradient'... doesn't have the same meaning in English. We should translate it and put in the brackets at the first time. After that, should use the English origin word.

vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
@tuananhhedspibk
Copy link
Contributor Author

@minhbk @aHungNguyenKhanh I have fixed my translation, could you guys review one more time for me? Thank a lot.

vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
vi/cheatsheet-deep-learning.md Outdated Show resolved Hide resolved
@minhntm
Copy link

minhntm commented Oct 25, 2019

LGTM

@tuananhhedspibk
Copy link
Contributor Author

Hi @damminhtien , could you review this translation 's content for the last time for me? If it's OK, i think we can request to merge this pull request.

@shervinea
Copy link
Owner

Thank you @tuananhhedspibk for all your work in this translation. Many thanks also to @rootonchair, @damminhtien, @aHungNguyenKhanh, @hoangvudat, @tt-anh-eole and @minhntm for your thorough review! Merging this PR right now.

@shervinea shervinea merged commit 3242439 into shervinea:master Apr 12, 2020
@shervinea shervinea changed the title [vi] Deep Learning [vi] cs-229-deep-learning Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reviewer wanted Looking for a reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants