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

translate advanced/static_quantization_tutorial.py #433

Merged
merged 1 commit into from
Mar 26, 2022

Conversation

des00
Copy link
Contributor

@des00 des00 commented Feb 13, 2022

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

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

static_quantization_tutorial.py 번역

@des00 des00 self-assigned this Feb 13, 2022
@des00
Copy link
Contributor Author

des00 commented Feb 13, 2022

원본 문서 문장들 뒤에 공백이 많았는데 IDE 이용해서 일괄적으로 삭제 했습니다.
리뷰하기 불편하시면 해당 공백들은 유지가 될 수 있도록 하겠습니다.

@des00
Copy link
Contributor Author

des00 commented Feb 13, 2022

@potatochips178 님, 일전에 번역하셨던 문서 참고하여 번역했습니다.
혹시 피드백 사항 있으시면 리뷰 부탁드립니다.

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.

사소한 리뷰 남겼습니다.
번역을 깔끔하게 잘해주셔서 읽는데도 부담이 별로 없었네요
고생하셨습니다!

MobileNet V2 메인 클래스
Args:
num_classes (int): 클래스 숫자
width_mult (float): 넓이 승수 - 이 수를 통해 각 계층의 채널 개수를 조절
Copy link
Contributor

Choose a reason for hiding this comment

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

넓이 승수라는 말은 영어 그대로 쓰는 것이 어떨까요?
Mobilenet논문에서 사용하는 단어임으로 번역을 하는 것보단 그대로 원어를 남겨두는 것이 낫다고 생각합니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저도 번역하기 애매해서 이전 번역자가 남긴 번역 그대로 옮겼는데
어차피 뒤에 설명도 있고 논문에도 있는 단어이니 그냥 단어 남겨놓는게 좋겠군요.

@des00 des00 force-pushed the static-quantization branch from 85eee9c to 591a7ef Compare February 16, 2022 11:57
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.

소소하게 리뷰를 남겼습니다.
보시고 반영이 필요한 부분만 반영해주세요. 감사합니다. 🙇

print('Size (MB):', os.path.getsize("temp.p")/1e6)
os.remove('temp.p')

3. 데이터셋과 DataLoader 정의하기
Copy link
Member

Choose a reason for hiding this comment

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

(Dataset과 DataLoader 클래스가 아닌 이미지넷) dataset과 data loader를 뜻하는거 같긴 한데, 각각을 위해 (torchvision.)dataset과 DataLoader를 사용하고 있으니 일관성을 위해 Dataset과 DataLoader 정의하기라고 하면 너무 억지일까요?;
일관성을 위해 data loader를 표현할 좋은 한국어가 있으면 둘 다 한국어로 쓰고 싶은데 뭔가 애매하게 걸려서 고민 끝에 답도 없으면서 의견 남겨봅니다 =_=

Copy link
Contributor Author

Choose a reason for hiding this comment

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

일관성있게 둘 다 영문으로 하는게 좋을거 같군요.

데이터셋이 흔히 쓰이는 말이라 한글로 남겨두긴 했는데 밑에 보면 소스코드에 Dataset 클래스도 있어서
Dataset이라고 남겨두는게 맞겠네요.
데이터로더는 이전에도 #278 에서
클래스 이름으로 쓰이는 경우가 많아서 용어집에도 추가를 하지 않은거 같더라고요.
두 개를 따로 떼서 생각하다 결과가 더 나빠진 듯하네요.

3. 데이터셋과 DataLoader 정의하기
----------------------------------

마지막 주요 설정 단계로서 학습과 테스트셋을 위한 DataLoader를 정의합니다.
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.

영어의 셋을 그냥 뒤에 붙여서 한글은 아예 다른 뜻이 되었군요.
학습과 테스트 데이터를 위한 이 제일 의미가 비슷한거 같으니 이걸로 바꾸겠습니다.



Next, we'll load in the pre-trained MobileNetV2 model. We provide the URL to download the data from in ``torchvision``
`here <https://github.com/pytorch/vision/blob/master/torchvision/models/mobilenet.py#L9>`_.
다음으로 사전에 학습된 MobileNetV2을 읽어 들입니다. ``torchvision`` 에서 데이터를 다운로드받을 수 있는 URL은
Copy link
Member

Choose a reason for hiding this comment

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

읽어 들입니다 보다는 불러옵니다 가 조금 더 매끄러울거 같은 생각에 사족을 붙여봅니다.

더불어, torchvision repo의 mobilenet.py가 변경되어서 해당 URL이 사라졌는데요, 요건 원본에도 아직 반영되어 있지 않아서 PR 날려보시는 것도 좋을 것 같습니다. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

불러옵니다로 바꾸는게 확실히 자연스럽네요.
원본 PR도 고민해 보겠습니다.

@des00 des00 force-pushed the static-quantization branch from 591a7ef to 3b585ae Compare March 3, 2022 13:59
@9bow
Copy link
Member

9bow commented Mar 26, 2022

👍

@9bow 9bow merged commit 4432145 into PyTorchKorea:master Mar 26, 2022
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