Skip to content

Conversation

@SanggiHong
Copy link

#13 이미 다른분이 이슈를 잡아놓으셨지만, 오랜기간 작업이 없어서 한번 해보았습니다. 괜찮을까요?

@SanggiHong
Copy link
Author

SanggiHong commented Oct 15, 2016

혹시 질서를 해치는 행동이였다면, 정말 죄송합니다. 말씀해주시면 close 하겠습니다.

@hunkim
Copy link
Collaborator

hunkim commented Oct 15, 2016

@SanggiHong 수고해주셔서 감사합니다. @totuta님에게 혹 리뷰 가능하신지 여쭈어 보았습니다.

@SanggiHong
Copy link
Author

네! 감사합니다.

@totuta
Copy link
Contributor

totuta commented Oct 15, 2016

죄송합니다. 오래전에 시작해놓고 마무리를 못하고 있었습니다. 제가 하던 부분과 비교하며 리뷰하도록 하겠습니다.

@totuta
Copy link
Contributor

totuta commented Oct 15, 2016

제가 리뷰를 하는 것 보다, 해당 페이지의 앞부분을 예전에 번역해 놓은 것이 있는데, 그것을 @SanggiHong 님께서 직접 참고하시는 것이 더 좋을 것 같네요.

https://github.com/totuta/tensorflow-kr/blob/master/g3doc/how_tos/variable_scope/index.md

이 페이지를 보시면 되겠습니다.

@SanggiHong
Copy link
Author

@totuta 혹시 커밋 내역을 하나로 합쳐주실 수 있으신가요? 참고함과 동시에 그곳에 제가 작업내역을 덧붙이면 어떨까요?

@totuta
Copy link
Contributor

totuta commented Oct 16, 2016

@SanggiHong 커밋 내역을 어떻게 하면 합칠 수 있나요? 제가 잘 몰라서..

@SanggiHong
Copy link
Author

@totuta http://ko.gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html 를 참조하시면 됩니다.
이미 깃헙에 올라간 커밋은 합칠 수 없으니, 새로 branch를 만드신 다음, 새 branch에서 합쳐서 깃헙에 push하시면 될 것 같아요.

@hunkim
Copy link
Collaborator

hunkim commented Oct 16, 2016

@totuta님 일단 pr 주시면 저희들이 merge 하고 @SanggiHong 님 pr 업데이트 해주시면 좋을듯 합니다. 두분 모두
감사합니다.
On Sun, 16 Oct 2016 at 12:42 PM, Sang Gi Hong notifications@github.com
wrote:

@totuta https://github.com/totuta
http://ko.gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html
를 참조하시면 됩니다.
이미 깃헙에 올라간 커밋은 합칠 수 없으니, 새로 branch를 만드신 다음, 새 branch에서 합쳐서 깃헙에 push하시면 될 것
같아요.


You are receiving this because you commented.

Reply to this email directly, view it on GitHub
#136 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3DV-TcyZzwT0glGdh-2wWmRdu1xV8bks5q0askgaJpZM4KXsnY
.

@SanggiHong
Copy link
Author

@hunkim
@totuta 님의 기존 작업에 덧붙이는 방식으로 해보았습니다. 리뷰를 부탁드려도 될까요?

@totuta
Copy link
Contributor

totuta commented Oct 22, 2016

죄송합니다. 어디서 리뷰하면 되나요? 제가 깃헙에 익숙치가 않아서..

@SanggiHong
Copy link
Author

SanggiHong commented Oct 23, 2016

처음부터 익숙한사람이 있을까요. 사과하시지 마셔요 :)
file changed 탭을 참고하시어 적절치 않은부분을 지적해주시면 될것 같습니다.

@totuta
Copy link
Contributor

totuta commented Nov 29, 2016

죄송합니다. 제가 시간을 내어서 작업을 계속해보려고 했지만, 도저히 시간이 나지 않네요. 제가 했던 것들은 그냥 참고로 사용하셔서 @SanggiHong 님께서 작업을 마무리하시는 것이 좋을 것 같습니다. 부탁드려요.

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