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

beginner_source/fgsm_tutorial.py 문법체크 #512

Merged
merged 1 commit into from Aug 3, 2022
Merged

beginner_source/fgsm_tutorial.py 문법체크 #512

merged 1 commit into from Aug 3, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jul 29, 2022

라이선스 동의

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

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

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

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

관련 이슈 번호

#513

PR 종류

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

PR 설명

beginner_source/fgsm_tutorial.py의 문법체크를 진행하였습니다.

기능을 호출 할 때마다 -> 기능을 호출할 때마다
-하다'는 앞의 명사와 붙여야 합니다.

정확도을 -> 정확도를
조사 오류입니다.

첫번째는 -> 첫 번째는
두번째는 -> 두 번째는
사물의 수나 양을 나타내어 체언을 꾸미는 수관형사는 뒤에 오는 명사 또는 의존 명사와 띄어 씁니다.

공격을 받는 -> 공격받는
굳이 조사 '을/를/이/가'를 쓰지 않아도 된다면 쓰지 않습니다.

복사 하였습니다. -> 복사하였습니다.
이 예의 '-하다'는 앞의 명사와 붙여야 합니다.

최대화하하는 -> 최대화하는
입력오류

잘못 되도록 -> 잘못되도록
'못되다'는 '성질이나 하는 짓이 악하거나 고약하다'는 뜻입니다.

@hyoyoung hyoyoung added the 컨트리뷰톤 오픈소스 컨트리뷰톤 관련 이슈/PR label Jul 29, 2022
@stella-oh
Copy link
Contributor

stella-oh commented Aug 2, 2022

띄어쓰기 수정으로 훨씬 가독성이 좋아졌습니다~ 👍

@garam24
Copy link
Contributor

garam24 commented Aug 3, 2022

한번에 여러 군데를 찾으셨네요!
정확한 문법을 따르도록 수정되니 문서가 보다 신뢰성 있는 것 같습니다.

수고 많으셨습니다!

Copy link
Member

@hyoyoung hyoyoung left a comment

Choose a reason for hiding this comment

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

good

@hyoyoung hyoyoung merged commit 416379e into PyTorchKorea:master Aug 3, 2022
bub3690 pushed a commit to bub3690/tutorial_kr_myrepo that referenced this pull request Aug 7, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
컨트리뷰톤 오픈소스 컨트리뷰톤 관련 이슈/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants