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

KO : edit ko/guide/variables.md #996

Closed
wants to merge 1 commit into from

Conversation

@dogemad
Copy link
Contributor

commented Sep 12, 2019

Corrected to the word "레이어" instead of the word "층".

I think it will convey better meaning when you use the word "레이어" rather than the word "층".

It is true that "층" is "레이어" in Korean, but I think it is more natural to use "레이어" considering context and word specificity.

thanks!

@tfdocsbot

This comment has been minimized.

Copy link
Collaborator

commented Sep 12, 2019

Reviewers added, please take a look.
@rickiepark, @cre8tor, @Choiuijin1125, @JKIsaacLee

When your review is finished, approve the pull request or include "LGTM" in your comment.

@googlebot googlebot added the cla: yes label Sep 12, 2019
@JKIsaacLee

This comment has been minimized.

Copy link
Contributor

commented Sep 12, 2019

Thanks for your nice suggestion. 👍
I think both words make sense, however, our translation glossary recommends translating 'layer' to '층'.
I'm not sure to agree this suggestion. I think it's up to other reviewers.

@Choiuijin1125

This comment has been minimized.

Copy link
Contributor

commented Sep 13, 2019

Hi @dogemad! thank you for your suggestion :)
As @JklsaacLee mentioned, both words make sense but we decided to use '층' rather than '레이어' to escape misunderstanding.
So layer is mostly translated as '층' in those documents.
If there are any other issues, you can discuss it in our mailing list(https://groups.google.com/a/tensorflow.org/forum/#!forum/docs-ko). :)
Thanks !!

@rickiepark

This comment has been minimized.

Copy link
Contributor

commented Sep 16, 2019

Thanks @dogemad
But we translate 'layer' as '층'. Please refer http://bit.ly/tf-docs-translation-glossary.

@dogemad

This comment has been minimized.

Copy link
Contributor Author

commented Sep 20, 2019

@JKIsaacLee @Choiuijin1125 @rickiepark thanks for your review! i close it!

@dogemad dogemad closed this Sep 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.