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_source/custom_ops_landing_page.rst 번역 #924

Closed
wants to merge 2 commits into from

Conversation

eunhye-1
Copy link

@eunhye-1 eunhye-1 commented Sep 8, 2024

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

이 PR로 무엇이 달라지는지 대략적으로 알려주세요.
기존에 없던 문서 번역 https://pytorch.org/tutorials/advanced/custom_ops_landing_page.html

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.

전반적으로 잘되어있는데 사소한 확인 사항이 몇가지 있습니다.
살펴보시고 수정 가능한 부분은 수정 부탁드립니다.

그러나 PyTorch에 새로운 사용자 지정 연산을 도입하여 ``torch.compile``, autograd 및 ``torch.vmap`` 와 같은
서브시스템에서 동작하도록 할 수 있습니다.
이렇게 하려면, 파이썬 `torch.library docs <https://pytorch.org/docs/stable/library.html>`_ 문서 또는
C++ ``TORCH_LIBRARY`` API를 통해 PyTorch에 사용자 지정 연산을 등록해야 합니다.
Copy link
Member

Choose a reason for hiding this comment

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

custom operation은 사용자 지정 연산보다
사용자 정의 연산이 조금 더 어울리지 않을까요?

튜토리얼과 이 페이지에서 다루지 않는 정보는 다음을 참조하시기 바랍니다.
`The Custom Operators Manual <https://docs.google.com/document/d/1_W62p8WJOQQUzPsJYa7s701JXt0qf2OfLub2sbkHOaU>`_
(정보를 문서 사이트로 옮기는 작업을 진행 중입니다.)
위의 튜토리얼 중 하나를 먼저 읽은 다음 사용자 지정 연산자 설명서를 참조로 사용하는 것이 좋습니다;
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
Member

Choose a reason for hiding this comment

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

이 줄의 공백은 필요 없는 듯 보입니다

@hyoyoung
Copy link
Member

최종 코멘트 후 2주가 지났습니다. 일정 기간 답변이 없는 경우 자동으로 닫고 있습니다.
필요한 내용이 있으면 내일까지 수정 부탁드리겠습니다

@hyoyoung hyoyoung closed this Oct 23, 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.

2 participants