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

/intermediate_source/char_rnn_generation_tutorial.py 오타수정 #498 #500

Merged
merged 1 commit into from
Aug 3, 2022

Conversation

stella-oh
Copy link
Contributor

@stella-oh stella-oh commented Jul 28, 2022

라이선스 동의

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

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

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

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

관련 이슈 번호

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

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

PR 종류

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

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

PR 설명

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

(1) 언어 형성(단어 또는 다른 고차원 구조로도 수행 될 수 있음) -> 수행될
(2) 우리는 출력을 다음 문자의 확률로 해석 합니다. -> 해석합니다.
(3) 학습은 일상적인 일입니다. - 몇번 train() 을 호출하고, -> 몇 번
(4) # 입력을 위한 처음부터 마지막 문자(EOS 제외)까지의 One-hot 행렬 -> (공백수정) blank space 2번 에서 1번으로 수정

@stella-oh stella-oh changed the title Update char_rnn_generation_tutorial.py /intermediate_source/char_rnn_generation_tutorial.py 오타수정 #498 Jul 28, 2022
@garam24
Copy link
Contributor

garam24 commented Jul 28, 2022

위에 라이선스 동의에서 체크 박스 표시가 반영되지 않은 것 같습니다!

라이선스 동의

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

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

@hyoyoung hyoyoung added the 컨트리뷰톤 오픈소스 컨트리뷰톤 관련 이슈/PR label Jul 28, 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.

라이선스 동의 해주세요

@stella-oh stella-oh requested a review from hyoyoung July 29, 2022 00:08
@stella-oh
Copy link
Contributor Author

라이선스 동의 완료했습니다 :)

Copy link
Contributor

@gorae17 gorae17 left a comment

Choose a reason for hiding this comment

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

몇 번 train() 을 호출하고 -> train() 을 몇 번 호출하고 로 바꾸면 더 자연스러울 것 같습니다.

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.

몇 번 train() 을 호출하고 -> train() 을 몇 번 호출하고 로 바꾸면 더 자연스러울 것 같습니다.

gorae17 님이 남겨주신 comment입니다 다음번 혹시 개선 pr 주실때 참고 부탁드립니다

good

@hyoyoung hyoyoung merged commit 48465c8 into PyTorchKorea:master Aug 3, 2022
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

4 participants