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

beginner_source/nlp/word_embeddings_tutorial.py, deep_learning_tutorial.py 문법, 오타 수정 #518

Merged
merged 7 commits into from
Aug 3, 2022

Conversation

sne12345
Copy link
Contributor

@sne12345 sne12345 commented Jul 29, 2022

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

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

1 ) 해야 할 지 -> 해야 할지
이 표현법을 어떠한 방식으로 연산해야 할 지 또한 큰 문제입니다. -> 이 표현법을 어떠한 방식으로 연산해야 할지 또한 큰 문제입니다.
2 ) 선형 대수학와 -> 선형 대수학과
PyTorch 와 대부분의 다른 딥러닝 프레임워크들은 고전적인 선형 대수학와 -> PyTorch 와 대부분의 다른 딥러닝 프레임워크들은 고전적인 선형 대수학과
3 ) 두개 -> 두 개
4 ) 더욱 더 -> 더욱더
5 ) 절대 값 -> 절댓값

추가 정보

1 ) 국립국어원 출처(https://www.korean.go.kr/front/onlineQna/onlineQnaView.do?&mn_id=&qna_seq=16583&pageIndex=13144)
Q. 해야할지를 붙여쓰는 건가요? 아님 띄어쓰는 건가요?
A. 안녕하십니까?
‘해야 할지’는 ‘하-+-어야 하다’에 추측에 대한 막연한 의문이 있는 채로 그것을 뒤 절의 사실이나 판단과 관련시키는 데 쓰는 연결 어미인 ‘-ㄹ지’가 붙은 형태입니다. 따라서 본용언과 보조 용언은 띄어 쓴다는 원칙에 따라 ‘해야 할지’로 띄어 씁니다.
3 ) 국립국어원 출처(https://korean.go.kr/front/onlineQna/onlineQnaView.do?mn_id=216&qna_seq=226419&pageIndex=1)
4 ) 국립국어원 출처(https://www.korean.go.kr/front/onlineQna/onlineQnaView.do?&mn_id=84&qna_seq=2857&pageIndex=14400)
5 ) 국립국어원 출처(https://twitter.com/urimal365/status/497293340245639168)

@sne12345 sne12345 changed the title beginner_source/nlp/word_embeddings_tutorial.py 문법 오류 수정 beginner_source/nlp/word_embeddings_tutorial.py 문법, 오타 수정 Jul 29, 2022
@sne12345 sne12345 changed the title beginner_source/nlp/word_embeddings_tutorial.py 문법, 오타 수정 beginner_source/nlp/word_embeddings_tutorial.py, deep_learning_tutorial.py 문법, 오타 수정 Jul 29, 2022
@hyoyoung hyoyoung added the 컨트리뷰톤 오픈소스 컨트리뷰톤 관련 이슈/PR label Jul 29, 2022
Copy link
Contributor

@terri1102 terri1102 left a comment

Choose a reason for hiding this comment

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

lgtm!🎉 다음부터는 한 PR에 한 파일만 수정하는 건 어떨까요?

Copy link
Contributor

@updaun updaun left a comment

Choose a reason for hiding this comment

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

와우! Commit을 오타 하나 수정할 때마다 하셨네요!? 정말 부지런하십니다! GOOD! 😄

Copy link
Contributor

@YunjinJo YunjinJo left a comment

Choose a reason for hiding this comment

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

LGTM

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 4694e44 into PyTorchKorea:master Aug 3, 2022
@hjvision96
Copy link
Contributor

많은 양을 찾아주셨네요! 좋습니다

bub3690 pushed a commit to bub3690/tutorial_kr_myrepo that referenced this pull request Aug 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
컨트리뷰톤 오픈소스 컨트리뷰톤 관련 이슈/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants