-
Notifications
You must be signed in to change notification settings - Fork 258
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
Conversation
원본 문서 문장들 뒤에 공백이 많았는데 IDE 이용해서 일괄적으로 삭제 했습니다. |
@potatochips178 님, 일전에 번역하셨던 문서 참고하여 번역했습니다. |
There was a problem hiding this 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): 넓이 승수 - 이 수를 통해 각 계층의 채널 개수를 조절 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넓이 승수라는 말은 영어 그대로 쓰는 것이 어떨까요?
Mobilenet논문에서 사용하는 단어임으로 번역을 하는 것보단 그대로 원어를 남겨두는 것이 낫다고 생각합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저도 번역하기 애매해서 이전 번역자가 남긴 번역 그대로 옮겼는데
어차피 뒤에 설명도 있고 논문에도 있는 단어이니 그냥 단어 남겨놓는게 좋겠군요.
85eee9c
to
591a7ef
Compare
There was a problem hiding this 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 정의하기 |
There was a problem hiding this comment.
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를 표현할 좋은 한국어가 있으면 둘 다 한국어로 쓰고 싶은데 뭔가 애매하게 걸려서 고민 끝에 답도 없으면서 의견 남겨봅니다 =_=
There was a problem hiding this comment.
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를 정의합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(학습과 테스트) 세트인데 학습 + 테스트셋 으로 보이는거 같습니다.
그냥 학습과 테스트를 위한
또는 학습과 테스트 데이터를 위한
같은 식은 어떨까요?
There was a problem hiding this comment.
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은 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
불러옵니다
로 바꾸는게 확실히 자연스럽네요.
원본 PR도 고민해 보겠습니다.
591a7ef
to
3b585ae
Compare
👍 |
라이선스 동의
변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.
더 자세한 내용은 기여하기 문서를 참고해주세요.
동의하시면 아래
[ ]
를[x]
로 만들어주세요.관련 이슈 번호
이 Pull Request와 관련있는 이슈 번호를 적어주세요.
이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )
PR 종류
이 PR에 해당되는 종류 앞의
[ ]
을[x]
로 변경해주세요.PR 설명
이 PR로 무엇이 달라지는지 대략적으로 알려주세요.
static_quantization_tutorial.py 번역