beginner_source/nlp/word_embeddings_tutorial.py, deep_learning_tutorial.py 문법, 오타 수정 - #518
Merged
Merged
Conversation
sne12345
changed the title
sne12345
changed the title
terri1102
reviewed
Aug 1, 2022
terri1102
left a comment
terri1102
left a comment
Contributor
There was a problem hiding this comment.
lgtm!🎉 다음부터는 한 PR에 한 파일만 수정하는 건 어떨까요?
updaun
approved these changes
Aug 2, 2022
updaun
left a comment
updaun
left a comment
Contributor
There was a problem hiding this comment.
와우! Commit을 오타 하나 수정할 때마다 하셨네요!? 정말 부지런하십니다! GOOD! 😄
Contributor
|
많은 양을 찾아주셨네요! 좋습니다 |
bub3690
pushed a commit
to bub3690/tutorial_kr_myrepo
that referenced
this pull request
Aug 7, 2022
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
6 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로 무엇이 달라지는지 대략적으로 알려주세요.
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)