beginner_source/dcgan_faces_tutorial.py 오탈자 및 내용 수정#1073
beginner_source/dcgan_faces_tutorial.py 오탈자 및 내용 수정#1073skt0725 wants to merge 1 commit intoPyTorchKorea:masterfrom
Conversation
|
문장 흐름을 자연스럽게 번역해주셔서 읽기 좋았습니다. 수고 많으셨습니다! |
| # 진짜 이미지들의 비교입니다 | ||
| # | ||
| # **학습하는 동안의 손실값들** | ||
| # **학습하는 동안의 손실 값들** |
There was a problem hiding this comment.
'손실값'도 한글 맞춤법 제50항(전문 용어 붙여 쓰기 허용)에 따라 가능한 표기라서, 다른 튜토리얼 일관성 생각하면 원래대로 둬도 괜찮을 것 같아요! 어떻게 생각하세요? 🙂
There was a problem hiding this comment.
다른 문서 (beginner_source/introyt/introyt1_tutorial.py) 참고해보니 말씀주신대로 붙여쓰기로 표기가 되어있는 것을 확인했습니다! 금일 미팅 후 붙여쓰기로 수정해서 다시 PR 올릴게요 감사드립니다!!
hyoyoung
left a comment
There was a problem hiding this comment.
전반적으로 잘되어있는데, 몇가지 수정사항 부탁드립니다
| # | ||
| # GAN이란 학습 데이터들의 분포를 학습한 뒤, 동일한 분포를 갖는 새로운 데이터를 | ||
| # 생성하도록 딥러닝 모델을 학습시키는 프레임워크입니다. | ||
| # 생성하도록 딥러닝 모델을 학습하는 프레임워크입니다. |
There was a problem hiding this comment.
이건 학습시키는이 더 맞지 않을까요? 트레이닝을 하는 주체를 봐야할거 같습니다
There was a problem hiding this comment.
좋은 피드백 감사합니다! 학습 주체는 굳이 따지자면 개발자, 혹은 'we'이고, 'GAN이라는 생성 모델을 학습하여 새로운 데이터를 생성한다' 라는 것을 설명하는 줄입니다. 따라서 '학습시키는' 이라고 표현하면 GAN이라는 프레임워크를 통해 또다른 생성 모델을 학습시키는 것이라고 오해할 여지가 있어 '학습하는'으로 수정하였는데 이 부분 논의해보면 좋을 것 같습니다!
| # - **Loss_G** - 생성자의 손실 값. :math:`log(D(G(z)))` | ||
| # - **D(x)** - 구분자가 데이터를 판별한 확률 값입니다. 처음에는 1에 가까운 값이다가, | ||
| # G가 학습할수록 0.5 값에 수렴하게 됩니다. | ||
| # - **D(G(z))** - 가짜데이터들에 대한 구분자의 출력 값입니다. 처음에는 0에 가까운 값이다가, |
There was a problem hiding this comment.
위에서는 다 가짜 데이터로 되어있으니 여기도 띄어쓰기 바꾸면 좋을거 같습니다
라이선스 동의
변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.
더 자세한 내용은 기여하기 문서를 참고해주세요.
동의하시면 아래
[ ]를[x]로 만들어주세요.관련 이슈 번호
이 Pull Request와 관련있는 이슈 번호를 적어주세요.
이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )
PR 종류
이 PR에 해당되는 종류 앞의
[ ]을[x]로 변경해주세요.PR 설명
이 PR로 무엇이 달라지는지 대략적으로 알려주세요.
오역, 오타 및 띄어쓰기 오류 수정하였습니다.