KEMBAR78
beginner_source/hta_trace_diff_tutorial.rst 번역 by uddk6215 · Pull Request #941 · PyTorchKorea/tutorials-kr · GitHub
Skip to content

Conversation

@uddk6215
Copy link
Contributor

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

  • 이슈 번호: #(940)

PR 종류

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

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

PR 설명

hta_trace_diff_tutorial.rst 파일을 전체 번역했습니다.

kernels that were added and removed from each group, along with the frequency
of each operator/kernel and the cumulative time taken by the operator/kernel.
때때로 사용자들은 코드 변경으로 인한 PyTorch 연산자와 CUDA 커널의 변화를 식별해야 할 필요가 있습니다.
이러한 요구를 지원하기 위해 HTA는 트레이스 비교 기능을 제공합니다. 이 기능을 통해 사용자는 두 세트의 트레이스 파일을 입력할 수 있는데,
Copy link
Contributor

Choose a reason for hiding this comment

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

사소한 부분이긴 하지만 가독성 측면에서 아래와 같이 표현한다면 어떨까요?

  • 이러한 요구를 지원하기 위해 -> 이를 위해

Copy link
Contributor Author

Choose a reason for hiding this comment

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

훨씬 보기 좋네요. 감사합니다.

@jason9865
Copy link
Contributor

jason9865 commented Sep 22, 2024

전반적으로 훌륭하게 번역해주신 것 같습니다!

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.

전반적으로 잘되어있습니다.
몇가지 확인사항을 남기니 수정 부탁드립니다.

종합적 분석을 이용한 트레이스 차이 분석
========================================

**Author:** `Anupam Bhatnagar <https://github.com/anupambhatnagar>`_
Copy link
Member

Choose a reason for hiding this comment

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

author는 저자로 변환 해주시고
저자 정보 아래에 번역자 정보 추가 부탁드립니다.


The `TraceDiff <https://hta.readthedocs.io/en/latest/source/api/trace_diff_api.html>`_ class
has the following methods:
`TraceDiff <https://hta.readthedocs.io/en/latest/source/api/trace_diff_api.html>`_ 클래스는 다음과 같은 메서드를 가지고 있습니다.
Copy link
Member

Choose a reason for hiding this comment

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

용어집에서 method는 메소드로 권장하고 있습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

모두 반영했습니다!

The `TraceDiff <https://hta.readthedocs.io/en/latest/source/api/trace_diff_api.html>`_ class
has the following methods:
`TraceDiff <https://hta.readthedocs.io/en/latest/source/api/trace_diff_api.html>`_ 클래스는 다음과 같은 메소드를 가지고 있습니다.
TraceDiff 클래스의 메서드:
Copy link
Member

Choose a reason for hiding this comment

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

위의 줄에서는 메소드이고 아래줄에서는 메서드입니다

@uddk6215 uddk6215 requested a review from hyoyoung October 9, 2024 12:52
The last two methods can be used to visualize various changes in frequency and
duration of CPU operators and GPU kernels, using the output of the
``compare_traces`` method.
마지막 두 메서드는 compare_traces 메서드의 출력을 사용하여 CPU 연산자와 GPU 커널의
Copy link
Member

Choose a reason for hiding this comment

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

여기도 메서드가 남아있습니다 확인 부탁드릴게요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

반영했습니다!

@uddk6215 uddk6215 requested a review from hyoyoung October 14, 2024 06:39
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.

good

@hyoyoung hyoyoung merged commit 9ce499f into PyTorchKorea:master Oct 15, 2024
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.

4 participants