intermediate_source/process_group_cpp_extension_tutorial.rst 번역 진행 by jenner9212 · Pull Request #764 · PyTorchKR/tutorials-kr

jenner9212 · 2023-09-02T04:12:48Z

제목 및 첫 문단 번역

라이선스 동의

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

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

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

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

PR 종류

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

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

PR 설명

이 PR로 무엇이 달라지는지 대략적으로 알려주세요.
intermediate_source/process_group_cpp_extension_tutorial.rst 파일 번역 진행 중입니다.

제목 및 첫 문단 번역

두 번째 문단 번역

세 번째 문단(step 1) 번역

네 번째 문단(step 2) 번역

다섯 번째 문단(step 3) 번역

여섯 번째 문단(step 4) 번역

띄어쓰기 추가

jih0-kim

리뷰 진행하였습니다.

jih0-kim · 2023-09-07T11:36:25Z

-Step 1: Implement a Subclass of ``ProcessGroup``
+클래스로 추상화합니다. 이후에는 원하는 서드파티 라이브러리를 사용하여 ``ProcessGroup``의 하위 클래스로 다양한 백엔드를 구현할 수 있습니다.
+파이토치 분산에는 세 가지 기본 백엔드인 ``ProcessGroupNCCL``, ``ProcessGroupGloo``, 그리고 ``ProcessGroupMPI``가 포함되어 있습니다.
+그러나 그러나 이 세 가지 백엔드 외에도 다른 통신 라이브러리(예: `UCC <https://github.com/openucx/ucc>`__, `OneCCL <https://github.com/oneapi-src/oneCCL>`__), 다른 유형의 하드웨어(예: `TPU <https://cloud.google.com/tpu>`__, `Trainum <https://aws.amazon.com/machine-learning/trainium/>`__), 


31번 라인의 시작에 같은 말이 두 번 입력된 것 같습니다.
그러나 그러나 -> 그러나

감사합니다! 확인 후 수정했습니다.

hyoyoung

전반적으로 잘되어있습니다

번역자 이름 부분이 없습니다. 다른 문서 참고해서 추가 부탁드립니다
tensor는 일반적으로 번역하지 않습니다
https://github.com/PyTorchKorea/tutorials-kr/blob/master/TRANSLATION_GUIDE.md

참고하셔서 수정부탁드립니다

hyoyoung · 2023-09-17T12:25:09Z

+첫 번째 단계는 대상 집합 통신 API를 재정의하고 사용자 정의 통신 알고리즘을 실행하는 ``ProcessGroup`` 하위 클래스를 구현하는 것입니다.
+확장 기능은 통신 결과의 미래 역할을 하는 ``Work`` 하위 클래스를 구현해야 하며, 이는 응용 프로그램 코드에서 비동기 실행을 허용합니다.
+확장 기능이 서드파티 라이브러리를 사용하는 경우, 해당 확장 기능은 ``ProcessGroupDummy`` 하위 클래스에서 헤더를 포함하고 라이브러리 API를 호출할 수 있습니다.
+아래의 두 코드는 ``dummy.h`` 및 ``dummy.cpp``의 구현을 보여줍니다. 전체 구현은 `더미 집합(dummy collectives) <https://github.com/mrshenli/dummy_collectives>`__ 리포지토리에서 확인하실 수 있습니다.


리포지토리는 저장소로 순화해도 좋을거 같습니다

번역자 추가하고 번역 수정 했습니다

hyoyoung

추가 피드백 확인 부탁드립니다

hyoyoung · 2023-09-30T12:03:58Z

+호출되므로 확장 기능도 파이썬에 생성자 API를 노출해야 합니다.
+다음 메서드를 추가함으로써 이 작업을 수행할 수 있습니다. 
+이 예제에서는 ``store``와 ``timeout``이 사용되지 않으므로 ``ProcessGroupDummy`` 인스턴스화 메서드에서 무시됩니다.
+그러나 실제 확장 기능은 랑데뷰를 수행하고 ``timeout`` 인수를 지원하기 위해 ``store``을 고려해야 합니다.


store 사용을 고려해야 합니다.
'using' 부분이 빠진거 같습니다

추가했습니다

hyoyoung · 2023-09-30T12:14:12Z

-``dummy.cpp``. See the `dummy collectives <https://github.com/mrshenli/dummy_collectives>`__
-repository for the full implementation.
+첫 번째 단계는 대상 집합 통신 API를 재정의하고 사용자 정의 통신 알고리즘을 실행하는 ``ProcessGroup`` 하위 클래스를 구현하는 것입니다.
+확장 기능은 통신 결과의 미래 역할을 하는 ``Work`` 하위 클래스를 구현해야 하며, 이는 응용 프로그램 코드ㄹ에서 비동기 실행을 허용합니다.


'코드ㄹ에서'는 오타인거 같습니다

results 부분이 빠져있는데
'확장 기능은 미래 통신 결과를 제공하는 Work 하위 클래스를 구현해야 하며' 이게 조금 더 자연스럽지 않을까요?

변경했습니다!

hyoyoung

여기 future관련 병기 표기 추가 부탁드립니다

hyoyoung · 2023-10-05T13:02:02Z

-``dummy.cpp``. See the `dummy collectives <https://github.com/mrshenli/dummy_collectives>`__
-repository for the full implementation.
+첫 번째 단계는 대상 집합 통신 API를 재정의하고 사용자 정의 통신 알고리즘을 실행하는 ``ProcessGroup`` 하위 클래스를 구현하는 것입니다.
+확장 기능은 미래 통신 결과를 제공하는 ``Work`` 하위 클래스를 구현해야 하며, 이는 응용 프로그램 코드에서 비동기 실행을 허용합니다.


여기는 맥락이 조금 더 필요한 부분이 있는데, 비동기 실행에서
future는 보통 어떤 context를 가지고 아무때나 실행될수 있고 결과를 return해주는 객체를 가르킵니다
미래라고 번역하기도 하는데 퓨쳐라고 읽기도합니다
'미래 통신 결과'는 '미래(future) 통신 결과'로 병기를 해두는게 더 이해하기 좋을거 같습니다

그렇군요! 수정하겠습니다

hyoyoung

good

jenner9212 added 7 commits September 2, 2023 12:48

first translation

2422ce9

제목 및 첫 문단 번역

second translation

1a67623

두 번째 문단 번역

third translation

6933fa3

세 번째 문단(step 1) 번역

fourth translation

ed8ff3d

네 번째 문단(step 2) 번역

fifth translation

2a25f2a

다섯 번째 문단(step 3) 번역

sixth translation

aee754e

여섯 번째 문단(step 4) 번역

Add space

20b9758

띄어쓰기 추가

jih0-kim reviewed Sep 7, 2023

View reviewed changes

hyoyoung requested changes Sep 17, 2023

View reviewed changes

fix review

3fa9349

jenner9212 requested a review from hyoyoung September 26, 2023 14:03

hyoyoung requested changes Sep 30, 2023

View reviewed changes

jenner9212 added 3 commits September 30, 2023 22:11

fix review2

d4b88d6

changed spacing

f2eca4e

fix translation

a0a245d

jenner9212 requested a review from hyoyoung September 30, 2023 13:24

hyoyoung requested changes Oct 5, 2023

View reviewed changes

fix review3

1403a87

jenner9212 requested a review from hyoyoung October 7, 2023 02:47

hyoyoung approved these changes Oct 8, 2023

View reviewed changes

hyoyoung merged commit 4144d34 into PyTorchKR:master Nov 26, 2023

Uh oh!

Conversation

jenner9212 commented Sep 2, 2023

라이선스 동의

관련 이슈 번호

PR 종류

PR 설명

Uh oh!

jih0-kim left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hyoyoung left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hyoyoung left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hyoyoung left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hyoyoung left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants