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

beginner_source/bettertransformer_tutorial.rst 번역 #916

Merged
merged 2 commits into from
Oct 15, 2024

Conversation

uddk6215
Copy link
Contributor

@uddk6215 uddk6215 commented Sep 5, 2024

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

beginner_source의 bettertransformer_tutorial.rst파일을 전체 번역하였습니다.

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**: `Michael Gschwind <https://github.com/mikekgfb>`__
**저자**: `마이클 그쉬빈드 <https://github.com/mikekgfb>`__
Copy link
Member

Choose a reason for hiding this comment

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

번역자 정보를 추가 해주세요

The fastpath feature works transparently for models based either directly on
PyTorch core ``nn.module`` or with torchtext.

이 튜토리얼에서는 PyTorch 1.12 릴리스의 일부로 Better Transformer (BT)를 소개합니다.
Copy link
Member

Choose a reason for hiding this comment

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

릴리스는 단어를 빼거나, 배포 혹은 버전으로 바꿔도 좋을거 같습니다

PyTorch core ``nn.module`` or with torchtext.

이 튜토리얼에서는 PyTorch 1.12 릴리스의 일부로 Better Transformer (BT)를 소개합니다.
여기서는 torchtext를 사용한 Production 추론에 Better Transformer를 적용하는 방법을 보여줍니다.
Copy link
Member

Choose a reason for hiding this comment

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

production은 소문자이므로
production 추론 일텐데요 상용 제품을 의미하는 것이니. 의역이 필요할 것 같습니다.
상용 수준 추론 정도로 하면 어떨까 싶은데 어떻게 생각하십니까?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

TRANSLATION_GUIDE.md에서 production을 따로 번역하지 않음을 참조한 것이었는데
상용 제품을 의미하는 것이군요! 수정했습니다.
뒤에 다시 한 번 production이 나오는 데, 이 부분도 고려해서 반영했습니다.


이 튜토리얼에서는 PyTorch 1.12 릴리스의 일부로 Better Transformer (BT)를 소개합니다.
여기서는 torchtext를 사용한 Production 추론에 Better Transformer를 적용하는 방법을 보여줍니다.
Better Transformer는 CPU와 GPU에서 고성능으로 트랜스포머 모델의 배포를 가속화하기 위한 Production으로 바로 적용가능한 fastpath입니다.
Copy link
Member

Choose a reason for hiding this comment

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

Better Transformer는 CPU와 GPU에서 고성능인 트랜스포머 모델 배포를 가속화할 수 있는, 상용 제품화가 바로 가능한 fastpath입니다.

문장의 구조가 복잡해서 두 문장으로 나누던지, 의역을 추가해서 원문 의미를 살려보는 것도 좋을 듯 합니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

좀 더 명확하게 두 문장으로 나누는 쪽으로 번역했습니다.


1.1 Load pretrained models
1.1 사전 훈련된 모델 로드
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.

리뷰 주신 것들 모두 좋은 피드백인 것 같아 모두 반영했습니다. 감사합니다.

@hyoyoung
Copy link
Member

참고로 이슈 번호 앞뒤에 ()를 제외해주면 링크로 바로 갈수 있게 됩니다

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 requested a review from 9bow October 6, 2024 11:45
@hyoyoung hyoyoung merged commit 83e6559 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.

3 participants