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

Issue 04 #156

Merged
merged 17 commits into from
Oct 23, 2018
Merged

Issue 04 #156

merged 17 commits into from
Oct 23, 2018

Conversation

karl6885
Copy link
Collaborator

#4 : 프로젝트 초반부에 문서 번역은 빠르게 끝냈지만 cheat sheet 번역과 코드 이해 부분에서 막혀서 PR 올리기 머뭇머뭇하고 있었습니다.. 앞으로도 추가 번역 필요한 부분 있다면 해서 올리도록 하겠습니다!

리뷰 부탁드립니다 :)

karl6885 and others added 9 commits August 27, 2018 15:19
Branch 생성 및 Result 전까지 번역 완료
AlphaGo Zero cheat sheet 번역 시작
오타 수정 및 본문 번역 완료
이후에는 코드 추가 설명 및 AlphaGo Zero cheat sheet 번역 진행 예정
Guide document에 맞춰 양식 수정 및 키워드 추가
Copy link
Collaborator

@visionNoob visionNoob left a comment

Choose a reason for hiding this comment

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

재미있는 내용이네요!

Copy link
Member

@MijeongJeon MijeongJeon left a comment

Choose a reason for hiding this comment

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

@karl6885 알파제로에 대한 이야기와 코드까지! 완벽합니다 👍
긴 글 깔끔히 정리해주셔서 감사합니다!

  • 코드 내부 주석은 혹시 번역을 일부러 안하신건지 확인 부탁드려요
  • "입니다" / "이다" 통일 부탁드릴게요~

…into issue_04

# Conflicts:
#	04_파이썬과_케라스를_이용한_알파제로_만들기.md
@karl6885
Copy link
Collaborator Author

@MijeongJeon 리뷰 반영하여 수정하였습니다!


![](https://upload.wikimedia.org/wikipedia/commons/a/ad/Connect_Four.gif)

<div style="text-align: center;font-size:80%">Connect4 하는 법. 이해를 돕기 위해 역자가 추가로 삽입하였습니다.</div>
Copy link
Collaborator

@mike2ox mike2ox Oct 22, 2018

Choose a reason for hiding this comment

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

134 ~ 136라인이 이해를 위한 추가 삽입이므로 인용구문(>) 으로 원문과 구분해놓으면 어떨까요?




<div style="text-align: center;font-size:80%">Metasquares 게임 화면. 이해를 돕기 위해 역자가 추가로 삽입하였습니다.</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분도 원글과 구분되도록 표시하면 어떨까요.(>)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

리뷰 감사드립니다! 반영하여 수정하였습니다 :)




*의역과 역자의 해석에 따른 추가적인 설명이 포함되어 원작자의 의도가 다르게 전해질 여지가 있으니 유의하시기 바랍니다. 만약 오역이나 오타를 발견하신다면 karl6885@gmail.com으로 메일 주시면 감사하겠습니다.*
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@mike2ox mike2ox left a comment

Choose a reason for hiding this comment

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

해커톤때 작업하신 글 같아서 읽어보았습니다. 좋은 번역 감사합니다 ㅎㅎ

@MijeongJeon MijeongJeon merged commit d5bd155 into master Oct 23, 2018
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.

None yet

4 participants