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

advanced/rpc_ddp_tutorial 번역 #382

Merged

Conversation

dajeongPark-dev
Copy link
Contributor

@dajeongPark-dev dajeongPark-dev commented Nov 15, 2021

Translate rpc_ddp_tutorial.rst and main.py

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

이 PR로 무엇이 달라지는지 대략적으로 알려주세요.

advanced/rpc_ddp_tutorial을 번역했고 rpc_ddp_tutorial.rst 파일과 rpc_ddp_tutorial파일 안에 있는 main.py를 수정했습니다.
번역 작업을 마친 후 'make html -noplot'을 통해 결과물을 확인하려고 했으나
어려움이 생겨서 그 부분까지 확인을 잘 하지 못했습니다. 하지만 최대한 reStructureText 규칙에 맞춰 수정하려고 노력했습니다.
부족한 부분이 있다면 피드백 부탁드립니다.

Translate rpc_ddp_tutorial.rst and main.py
Copy link
Contributor

@puhuk puhuk left a comment

Choose a reason for hiding this comment

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

리뷰 남깁니다.

advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
typo correction from line133
Copy link
Member

@hyoyoung hyoyoung left a comment

Choose a reason for hiding this comment

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

전체적으로 backward는 역방향으로 바꿔주세요

advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@Taeyoung96 Taeyoung96 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 몇 가지 리뷰를 남깁니다. :)

advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial/main.py Outdated Show resolved Hide resolved
Modified the translation based on reviews.
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial.rst Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial/main.py Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial/main.py Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial/main.py Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial/main.py Outdated Show resolved Hide resolved
advanced_source/rpc_ddp_tutorial/main.py Outdated Show resolved Hide resolved
@des00
Copy link
Contributor

des00 commented Dec 17, 2021

@dajeongPark-dev
혹시 리뷰 확인하시고 수정 가능하신지요?
피드백이 없으면 close 처리하고자 합니다.

@dajeongPark-dev
Copy link
Contributor Author

@dajeongPark-dev 혹시 리뷰 확인하시고 수정 가능하신지요? 피드백이 없으면 close 처리하고자 합니다.

늦은 피드백 죄송합니다... 빠르게 수정하도록 하겠습니다!

Incomplete _considering about how to solve the 'optimizer'
Fix translation errors.
@des00
Copy link
Contributor

des00 commented Dec 18, 2021

@dajeongPark-dev
밤늦은 시간까지 업데이트하시느라 고생이 많으시군요.
같은 수업을 들으시는 몇몇 분들의 불성실함과 비교되는 훌륭한 작업물입니다.
번역하신 부분 리뷰하며 저도 많이 배우고 있습니다.

혹시 수정이 완료되었으면 기존에 리뷰하셨던 분들에게
@user_id로 호출하여 수정한 부분 재리뷰를 청해보시는 것은 어떨런지요.
지속적으로 피드백을 주고 받는 만큼 얻어가시는 것이 많으리라 확신합니다.

@dajeongPark-dev
Copy link
Contributor Author

dajeongPark-dev commented Dec 18, 2021

@des00
번역 작업이 처음이라 쉽지는 않았지만 많은 분들께서 리뷰도 해주시고 원활한 작업 진행을 위해 옆에서 힘써주심에 많은 것을 배워가는 시간이었습니다. 응원과 격려에 감사드립니다!

그리고 튜토리얼 번역 관리하시는 데에 어려움을 드리게 되어 죄송하다는 말씀 전해드립니다. 코로나로 인해 이번 학기도 비대면으로 진행되었고 학기가 마무리되어가는 시점에서 파이토치 튜토리얼 번역을 진행하셨던 학우분들을 모두 알 수는 없는 상황이지만 제가 조금이라도 도움이 될 수 있는 부분이 있다면 성심성의껏 참여하겠습니다.

현재 리뷰해 주신 부분들을 다 수정한 상태이고 마지막으로 한 번 더 검토 중이었습니다! 여러 번 봐도 놓치는 부분이 있을 때도 있어서요 :) 오늘 안으로 마지막 검토를 마치고 알려주신 방법으로 재리뷰 부탁드리도록 하겠습니다! 늦은 시간에 감사합니다.

Fix translation errors
Fix a translation error
@dajeongPark-dev
Copy link
Contributor Author

@puhuk
안녕하세요! 이번에 rpc_ddp_tutorial 번역을 거의 다 완료하여 재리뷰를 부탁드리게 되었습니다.
저번에 피드백 주신 backward pass는 역방향 전달로 수정하게 되었습니다.

감사합니다!

@dajeongPark-dev
Copy link
Contributor Author

@Taeyoung96
안녕하세요! 이번에 rpc_ddp_tutorial 번역을 거의 다 완료하여 재리뷰를 부탁드리게 되었습니다.
저번에 주신 피드백을 참고하여 전반적으로 수정했습니다 :)

감사합니다!

Fix a translation error
@dajeongPark-dev
Copy link
Contributor Author

@hyoyoung
안녕하세요! 이번에 rpc_ddp_tutorial 번역을 거의 다 완료하여 재리뷰를 부탁드리게 되었습니다.
저번에 주신 피드백을 참고하여 전반적으로 수정했습니다 :)

그리고 초반에 optimizer를 최적화로 번역하면 어떨지 조언을 해주셨는데 그 부분에 대해서는 최적화가 어울릴 것라고 판단이 되어 그렇게 수정을 했지만 이후에 나오는 부분에서는 번역 용어집을 참고해보았을 때도 음차 표기 방식을 이용한다고 되어 있어 옵티마이저로 변경해 놓았습니다!

감사합니다!

@puhuk
Copy link
Contributor

puhuk commented Dec 20, 2021

@dajeongPark-dev
고생하셨습니다!

@Taeyoung96
Copy link
Contributor

@dajeongPark-dev
확인했습니다! 고생하셨습니다 :)

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.

@puhuk @Taeyoung96 @hyoyoung
다시 리뷰 요청드립니다. 특이 사항이 없으시면 차주 중 반영하려고 합니다.

@puhuk
Copy link
Contributor

puhuk commented Jan 10, 2022

LGTM 👍

Copy link
Contributor

@Taeyoung96 Taeyoung96 left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@9bow 9bow merged commit efe11eb into PyTorchKorea:master Jan 10, 2022
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

6 participants