Skip to content

Conversation

@jessemin
Copy link
Collaborator

라이선스 동의

변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.

더 자세한 내용은 기여하기 문서를 참고해주세요.

동의하시면 아래 [ ][x]로 만들어주세요.

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

관련 이슈 번호

이 Pull Request와 관련있는 이슈 번호를 적어주세요.

이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )

PR 종류

이 PR에 해당되는 종류 앞의 [ ][x]로 변경해주세요.

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

PR 설명

이 PR로 무엇이 달라지는지 대략적으로 알려주세요.
ONNX에 관한 튜토리얼인 EXPORTING A MODEL FROM PYTORCH TO ONNX AND RUNNING IT USING ONNX RUNTIME (https://tutorials.pytorch.kr/advanced/super_resolution_with_onnxruntime.html)를 번역하였습니다.

Comment on lines 14 to 15
본 튜토리얼을 위해서는 `ONNX <https://github.com/onnx/onnx>`__
and `ONNX Runtime <https://github.com/microsoft/onnxruntime>`__ 를 설치해야합니다.
Copy link
Member

Choose a reason for hiding this comment

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

아래와 같은 번역은 어떨까요?

이 튜토리얼을 진행하기 위해서는 ONNX <https://github.com/onnx/onnx>__ 와
ONNX Runtime <https://github.com/microsoft/onnxruntime>__ 설치가 필요합니다.

``NOTE``: This tutorial needs PyTorch master branch which can be installed by following
the instructions `here <https://github.com/pytorch/pytorch#from-source>`__
``노트``: 본 튜토리얼은 PyTorch의 master 브랜치를 필요로하며 `링크 <https://github.com/pytorch/pytorch#from-source>`__ 에서
Copy link
Member

@9bow 9bow Jan 30, 2020

Choose a reason for hiding this comment

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

찾아보니 기존에는 (widget이 아닌) Note:그대로 두거나, 또는 참고: 정도로 두었는데요, 생각난 김에 하나로 통일하는 것도 좋을 것 같습니다.
의견주시면 반영토록 하겠습니다. :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

우선 "참고:"로 수정하였습니다. 위젯이 아니기 때문에 한글로 번역할 수 있으면 일관성이 있을 것 같아 우선 제안해주신 "참고"로 진행하였는데, 필요하면 추후에 더 의논해봐도 좋을 것 같습니다.

Copy link
Member

Choose a reason for hiding this comment

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

네, 감사합니다.
기존에는 한정된 인원들만 번역에 참여했기 때문에 (잘 관리되지는 않았지만) Google Drive를 통해서 단어들을 공유하였는데요, 이런 식으로 Wiki를 이용하여 관리해보는 것이 필요할까요?

Comment on lines 43 to 46
# 아래는
# `링크
# <https://github.com/pytorch/examples/blob/master/super_resolution/model.py>`__
# 에서 구현된 것을 그대로 가져온 모델입니다:
Copy link
Member

Choose a reason for hiding this comment

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

아래와 같은 번역은 어떨까요?

아래는 PyTorch 예제의
모델 <https://github.com/pytorch/examples/blob/master/super_resolution/model.py>__
을 그대로 가져온 것입니다:

Comment on lines 148 to 149
# 모델이 타당한 스키마를 가지고 있는지를 체크합니다.
# ONNX 그래프의 타당성은 모델의 버전, 그래프 구조, 노드들, 그리고 입력값과 출력값들을 모두 체크하여
Copy link
Member

Choose a reason for hiding this comment

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

valid유효한 정도로 번역하여 유효한 스키마(valid schema)ONNX 그래프의 유효성(validity) 정도는 어떨까요?

Copy link
Member

@9bow 9bow left a comment

Choose a reason for hiding this comment

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

사족과도 같은 댓글들을 추가했습니다.
참고 부탁드리며, 변경이 필요없으거나 완료되셨을 때 알려주시면 이 PR을 병합하도록 하겠습니다.
감사합니다.

@jessemin
Copy link
Collaborator Author

빠른 리뷰와 코멘트들 감사합니다. 수정사항 반영한 뒤에 알려드리도록 하겠습니다.

@jessemin
Copy link
Collaborator Author

jessemin commented Feb 1, 2020

@9bow 좋은 제안들 감사합니다. 수정사항 반영하였습니다.

Copy link
Member

@9bow 9bow left a comment

Choose a reason for hiding this comment

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

반영 감사합니다! (_ _)

@9bow
Copy link
Member

9bow commented Feb 2, 2020

앗, 반영하려고 보니 가장 마지막 커밋만 반영된 것으로 나오네요.
원본 파일을 그대로 반영하도록 하겠습니다.

@9bow
Copy link
Member

9bow commented Feb 3, 2020

빌드 및 배포 결과물을 https://tutorials.pytorch.kr/advanced/super_resolution_with_onnxruntime.html 에서 확인하실 수 있습니다. ^^

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