Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

recipes_source/mobile_perf.rst 번역 #421

Merged
merged 1 commit into from
Jan 24, 2022
Merged

Conversation

des00
Copy link
Contributor

@des00 des00 commented Jan 8, 2022

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

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

mobile_perf.rst 번역

@des00 des00 requested review from 9bow and codingbowoo January 8, 2022 14:50
@des00
Copy link
Contributor Author

des00 commented Jan 8, 2022

@hongsam123님,
일전에 올리셨던 번역 참조하여 새롭게 번역했습니다.
가능하시다면 리뷰 부탁드립니다.

@9bow 9bow requested review from creduo, hrxorxm and Taeyoung96 January 9, 2022 11:50
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.

고생하셨습니다, 리뷰 남깁니다.

``torch.quantization.fuse_modules`` fuses a list of modules into a single module.
It fuses only the following sequence of modules:
``torch.quantization.fuse_modules`` 은 모듈들의 리스트를 하나의 모듈로 결합합니다.
이것은 아래 순서의 모듈들만 결합시킵니다:
Copy link
Contributor

Choose a reason for hiding this comment

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

sequence 는 순서 보다 결합, 연결 등의 의미로 사용된 것 같습니다. 아래 제안드립니다.

아래 모델들을 결합시킵니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

해당 함수의 레퍼런스를 찾아보니 아래와 같이 conv, bn 이렇게 모듈 종류와 순서가 정확해야
하나의 모듈로 결합을 한다고 나와 있네요.
https://pytorch.org/docs/stable/torch.quantization.html#torch.quantization.fuse_modules

밑의 모듈들도 순서대로 나열했을 때 결합 가능한 모듈들을 나열한 리스트라서
순서라는 말은 놔둬야 된다고 봅니다.

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.

유스케이스라는 단어가 가끔 쓰이는데 사용 사례로 번역하는 것이 어떤가 제안드립니다.
번역하시느라 고생하셨습니다. 👍

Copy link
Member

@codingbowoo codingbowoo left a comment

Choose a reason for hiding this comment

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

소소한 리뷰를 남깁니다 : ) 번역하시느라 고생 많으셨습니다! 👍 👍
(앗... 조금 더 일찍 리뷰를 남겼으면 좋았을 뻔 했군요)

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.

다른 분들께서 리뷰를 잘 해주셔서... 대세에 지장없는, 소소한 리뷰만 남깁니다. ^^;
다음 주 중으로 병합하도록 하겠습니다. 감사합니다!

translate mobile_pref, recipies_index
@9bow 9bow linked an issue Jan 23, 2022 that may be closed by this pull request
@9bow 9bow merged commit 620d28b into PyTorchKorea:master Jan 24, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

recipes_source/mobile_perf.rst 번역 예정
5 participants