'PyTorch가 무엇인가요?' 표현 개선 #78

itsnamgyu · 2020-07-06T09:00:06Z

라이선스 동의

기여하기 문서를 확인하였으며, 본 PR 내용에 BSD 3항 라이선스가 적용됨에 동의합니다.

PR 종류

오탈자를 수정하거나 번역을 개선하는 기여
번역되지 않은 튜토리얼을 번역하는 기여
공식 튜토리얼 내용을 반영하는 기여
위 종류에 포함되지 않는 기여

PR 설명

가독성을 위해 몇가지 표현을 수정했습니다. (컨트리뷰톤 지원자 사전 컨트리뷰션)

beginner_source/blitz/tensor_tutorial.py

가독성을 위해 몇가지 표현을 수정했습니다.

beginner_source/blitz/tensor_tutorial.py

9bow · 2020-07-06T14:39:20Z

beginner_source/blitz/tensor_tutorial.py

+# (CPU상의) Torch Tensor와 NumPy 배열은 메모리상 저장 공간을 공유하기 때문에,
 # 하나를 변경하면 다른 하나도 변경됩니다.


요건 The Torch Tensor and NumPy array will share their underlying memory locations (if the Torch Tensor is on CPU), 라고 되어 있던 것을 (Torch Tensor가 CPU 상에 있다면) Torch Tensor와 NumPy 배열은 메모리 공간을 공유하기 때문에,라는 의도에서 위와 같이 번역했던 것 같습니다. 제안해주신 내용이 더 나을지 판단이 어려운데요, @creduo 님, @itsnamgyu 님 의견 부탁드립니다.
덧붙여서, CPU 상과 같이 띄어쓴 것은 아래 첨부 이미지와 같이 부산대학교 맞춤법 검사기의 도움을 받았었습니다. :)

아하 부산대학교 검사기에 맞추겠습니다! (저는 국립국어원 자료를 참고했는데 논란의 여지가 있네요... 앞으로 부산대학교로 통일하겠습니다🙆‍♂️)

https://www.korean.go.kr/front/onlineQna/onlineQnaView.do?mn_id=216&qna_seq=1137

@9bow 님 말씀대로 explicit하게 (Torch Tensor가 CPU 상에 있다면) Torch Tensor와 NumPy 배열은 메모리 공간을 공유하기 때문에,하는 것이 가장 깔끔해보입니다. 일단 그렇게 수정해보겠습니다.

아, 국어국립원에도 관련한 내용이 있었군요.
(조금 어거지로) 구체적인 사물의 위나 위쪽을 나타내는 경우로 봐도 되지 않을까요? ^^;;

9bow · 2020-07-06T14:44:07Z

beginner_source/blitz/tensor_tutorial.py

 # NumPy 배열을 Torch Tensor로 변환하기
 # ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-# NumPy(np) 배열을 변경하면 Torch Tensor의 값도 자동 변경되는 것을 확인해보세요.
+# NumPy 배열을 변경하면 Torch Tensor의 값도 자동 변경되는 것을 확인해보세요.


요건 원문(See how changing the np array changed the Torch Tensor automatically)에 np로만 기재되어 있고, NumPy 를 처음으로 불러와서(import numpy as np) 사용하고 있어(np.ones(5)) NumPy로 쓰고 (np)를 덧붙였는데요, PyTorch를 처음 사용하시는 분들께서 NumPy에 얼마나 익숙하신지 몰라 선뜻 생략하지 못한 부분입니다.
(np) 부분이 어색하셨다면 원문에 맞춰 np 배열(NumPy array) 정도는 어떨까 싶은데요, @itsnamgyu 님, @creduo 님, 의견 부탁드립니다. :)

개인적으로 np=NumPy를 기본적으로 알고 있지 않을까 해서 생략했는데, 처음 접하시는 분들에게 혼선을 줄 수 있을 것 같네요!
워딩은 np (NumPy) 배열로 해서 업데이트해보겠습니다.

beginner_source/blitz/tensor_tutorial.py

9bow

감사합니다. 전체적으로 훨씬 읽기 수월해진 것 같습니다. @creduo 님께서 얘기해주신 것처럼 원문을 함께 참고해주시면 더 좋을 것 같습니다. ^^
덧글의 내용들은 참고로만 확인 부탁드리며, 확인 완료되시면 알려주시기를 부탁드립니다.
감사합니다!

Co-authored-by: Junghwan Park <reserve.dev@gmail.com>

Co-authored-by: Junghwan Park <reserve.dev@gmail.com> - CPU 상에 띄어쓰기 추가 (통일) etc.

itsnamgyu · 2020-07-07T07:37:28Z

정말 구체적인 피드백 감사합니다! @9bow 님께서 제안해주신대로 거의 모두 반영해서 커밋 추가했습니다. np (NumPy) 배열 표현을 조금 바꿨습니다 (댓글에 👀표시)

9bow

감사합니다!

9bow · 2020-07-09T13:23:07Z

@creduo 님,
추가 리뷰 없으시면 merge하려고 합니다. 괜찮으신가요?

9bow · 2020-07-11T15:07:03Z

https://tutorials.pytorch.kr/beginner/blitz/tensor_tutorial.html#sphx-glr-beginner-blitz-tensor-tutorial-py 에 반영되었습니다. :)

Co-authored-by: Junghwan Park <reserve.dev@gmail.com> - CPU 상에 띄어쓰기 추가 (통일) etc.

itsnamgyu changed the title ~~Minor updates to tensor_tutorial.py~~ tensor_tutorial.py 가독성 수정 Jul 6, 2020

itsnamgyu changed the title ~~tensor_tutorial.py 가독성 수정~~ 'PyTorch가 무엇인가요?' 표현 개선 Jul 6, 2020

creduo reviewed Jul 6, 2020

View reviewed changes

beginner_source/blitz/tensor_tutorial.py Show resolved Hide resolved

itsnamgyu force-pushed the tensor_tutorial_fix branch from 2f85dd9 to d281e35 Compare July 6, 2020 10:13

itsnamgyu requested a review from creduo July 6, 2020 10:14

Minor updates to tensor_tutorial.py

bbb35c6

가독성을 위해 몇가지 표현을 수정했습니다.

itsnamgyu force-pushed the tensor_tutorial_fix branch from d281e35 to bbb35c6 Compare July 6, 2020 10:14

9bow reviewed Jul 6, 2020

View reviewed changes

beginner_source/blitz/tensor_tutorial.py Outdated Show resolved Hide resolved

9bow reviewed Jul 6, 2020

View reviewed changes

beginner_source/blitz/tensor_tutorial.py Outdated Show resolved Hide resolved

9bow requested changes Jul 6, 2020

View reviewed changes

itsnamgyu added a commit to itsnamgyu/PyTorch-tutorials-kr that referenced this pull request Jul 7, 2020

tensor_tutorial.py 리뷰 사항 추가 (PyTorchKorea#78)

c27ef23

Co-authored-by: Junghwan Park <reserve.dev@gmail.com>

itsnamgyu added a commit to itsnamgyu/PyTorch-tutorials-kr that referenced this pull request Jul 7, 2020

tensor_tutorial.py 리뷰 사항 추가 (PyTorchKorea#78)

cae948c

Co-authored-by: Junghwan Park <reserve.dev@gmail.com>

itsnamgyu force-pushed the tensor_tutorial_fix branch from c27ef23 to cae948c Compare July 7, 2020 07:26

itsnamgyu added a commit to itsnamgyu/PyTorch-tutorials-kr that referenced this pull request Jul 7, 2020

tensor_tutorial.py 리뷰 사항 추가 (PyTorchKorea#78)

2378d90

Co-authored-by: Junghwan Park <reserve.dev@gmail.com> - CPU 상에 띄어쓰기 추가 (통일) etc.

itsnamgyu force-pushed the tensor_tutorial_fix branch from cae948c to 2378d90 Compare July 7, 2020 07:30

tensor_tutorial.py 리뷰 사항 추가 (PyTorchKorea#78)

cc6b6cb

Co-authored-by: Junghwan Park <reserve.dev@gmail.com> - CPU 상에 띄어쓰기 추가 (통일) etc.

itsnamgyu force-pushed the tensor_tutorial_fix branch from 2378d90 to cc6b6cb Compare July 7, 2020 07:33

itsnamgyu requested a review from 9bow July 7, 2020 07:39

9bow approved these changes Jul 8, 2020

View reviewed changes

9bow merged commit 555973a into PyTorchKorea:master Jul 11, 2020

itsnamgyu added a commit to itsnamgyu/PyTorch-tutorials-kr that referenced this pull request Aug 2, 2020

tensor_tutorial.py 리뷰 사항 추가 (PyTorchKorea#78)

7b25334

Co-authored-by: Junghwan Park <reserve.dev@gmail.com> - CPU 상에 띄어쓰기 추가 (통일) etc.

		# (CPU상의) Torch Tensor와 NumPy 배열은 메모리상 저장 공간을 공유하기 때문에,
		# 하나를 변경하면 다른 하나도 변경됩니다.

'PyTorch가 무엇인가요?' 표현 개선 #78

'PyTorch가 무엇인가요?' 표현 개선 #78

Uh oh!

Conversation

itsnamgyu commented Jul 6, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

라이선스 동의

PR 종류

PR 설명

Uh oh!

Uh oh!

Uh oh!

9bow Jul 6, 2020

Choose a reason for hiding this comment

Uh oh!

itsnamgyu Jul 7, 2020

Choose a reason for hiding this comment

Uh oh!

itsnamgyu Jul 7, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

9bow Jul 8, 2020

Choose a reason for hiding this comment

Uh oh!

9bow Jul 6, 2020

Choose a reason for hiding this comment

Uh oh!

itsnamgyu Jul 7, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

9bow left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

itsnamgyu commented Jul 7, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

9bow left a comment

Choose a reason for hiding this comment

Uh oh!

9bow commented Jul 9, 2020

Uh oh!

9bow commented Jul 11, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

itsnamgyu commented Jul 6, 2020 •

edited

Loading

itsnamgyu Jul 7, 2020 •

edited

Loading

itsnamgyu Jul 7, 2020 •

edited

Loading

9bow left a comment •

edited

Loading

itsnamgyu commented Jul 7, 2020 •

edited

Loading