Resnext translation#11
Open
JunyongKang wants to merge 5 commits into
Open
Conversation
petanerd
approved these changes
Jul 29, 2022
| Resnext 모델은 논문 [Aggregated Residual Transformations for Deep Neural Networks]에서 제안되었습니다. (https://arxiv.org/abs/1611.05431). | ||
| 이중 두가지 버전의 모델 성능은 아래와 같습니다. 각 모델의 레이어 개수는 각 50, 101개입니다. | ||
| resnet50과 resnext50의 아키텍처 차이는 논문의 Table 1을 참고하십시오. | ||
| ImageNet 데이터셋에 대한 사전훈련된 모델의 에러(성능)은 아래 표와 같습니다. |
There was a problem hiding this comment.
모델 설명의 본문에서는 '에러', 해당 내용을 가르키는 아래 표에서는 '오류'로 작성 되어있습니다.
외래어 표기법 등에 따라 둘 중에 하나로 통일하는 것이 좋지 않을까 생각 됩니다 :)
yh00214
reviewed
Jul 30, 2022
Comment on lines
+21
to
+23
| 모든 사전훈련된 모델은 입력 이미지가 같은 방식으로 정규화되었다고 가정합니다. | ||
| 즉, 미니배치(mini-batch)의 3채널 RGB 이미지들은 `(3 x H x W)`의 shape을 가지며, `H`와 `W`는 최소 `224`이상이어야 하며, 각 이미지들은 `[0, 1]`의 범위에서 로드되어야 하며, 그 다음 `mean = [0.485, 0.456, 0.406]` 과 `std = [0.229, 0.224, 0.225]`를 이용해 정규화되어야 합니다. | ||
| 아래 예시 코드가 있습니다. |
There was a problem hiding this comment.
- 22번째 줄 문장이 원문에서 두 문장으로 된 것을 합친 것으로 보입니다. TRANSLATION_GUIDE에 의거하여 문장 단위는 가급적 유지할 수 있도록, 문장을 두 문장으로 나누는 편이 좋을 것 같습니다.
- 22번째 줄의 '각 이미지들은
[0, 1]의 범위에서 로드되어야 하며'의 경우, 기존 다른 문서의 PR에서 로드되어야 한다는 말의 의미를 설명하는 게 좋겠단 리뷰를 받아들여 해당 부분의 설명을 추가한 사례가 있습니다. 현재 문서에도 해당 내용을 추가해도 좋을 것 같다고 생각합니다.
PyTorchKR/hub-kr@df0adb6
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
라이선스 동의
변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.
더 자세한 내용은 기여하기 문서를 참고해주세요.
동의하시면 아래
[ ]를[x]로 만들어주세요.관련 이슈 번호
이 Pull Request와 관련있는 이슈 번호를 적어주세요.
이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )
PR 종류
이 PR에 해당되는 종류 앞의
[ ]을[x]로 변경해주세요.PR 설명
이 PR로 무엇이 달라지는지 대략적으로 알려주세요.