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.

Sign up for GitHub

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

Jump to bottom

Char rnn classification tutorial typo correction #611

Merged
merged 6 commits into from
Dec 2, 2022
Merged

Char rnn classification tutorial typo correction #611

merged 6 commits into from
Dec 2, 2022

Conversation

garlicvread
Copy link
Contributor

@garlicvread garlicvread commented Oct 22, 2022 •
edited
Loading

라이선스 동의

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

관련 이슈 번호

PR 종류

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

PR 설명

  • 컨트리뷰터 정보 추가
  • 누락된 마침표 추가
    • 16, 195(beginner_source/saving_loading_models.py)
    • 535, 536, 537(intermediate_source/char_rnn_classification_tutorial.py)
  • 띄어쓰기 및 조사 수정

raymond added 6 commits October 22, 2022 15:20
saving_loading_models.py torchScript 미번역분 번역
7748fe7 
### Motivation
- TorchScript 부분 미번역 문단

### Key Changes
- TorchScript 미번역 문단 국문 번역
- 코드 주석 번역 추가

### To Reviewers
- 오픈소스 첫 기여라 재밌습니다 ㅎㅎ
오타 수정: Line 240, 기 -> 삭제
5af6ba3
오타 수정: Line 21, 올 -> 위치 수정
c6a7132
컨트리뷰터 정보 추가
9a78461
띄어쓰기 수정
4eea1af
띄어쓰기 및 오타 수정
4bfea8b
@garlicvread garlicvread marked this pull request as ready for review October 22, 2022 13:49
garlicvread
garlicvread commented Oct 22, 2022
View reviewed changes
Copy link
Contributor Author

@garlicvread garlicvread left a comment

Choose a reason for hiding this comment

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

PR을 조금 실수한 것 같은데, 수정해서 다시 커밋하는 것이 좋을까요?

garlicvread
garlicvread commented Oct 22, 2022
View reviewed changes
Copy link
Contributor Author

@garlicvread garlicvread left a comment

Choose a reason for hiding this comment

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

수정완료

garlicvread
garlicvread commented Oct 22, 2022
View reviewed changes
Copy link
Contributor Author

@garlicvread garlicvread left a comment

Choose a reason for hiding this comment

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

수정 완료

garlicvread
garlicvread commented Oct 22, 2022
View reviewed changes
Copy link
Contributor Author

@garlicvread garlicvread left a comment

Choose a reason for hiding this comment

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

컨트리뷰터 정보 추가했습니다. 그런데 이게 아까 오후에 한 내용이라 커밋을 수정해야하나 고민 중입니다.

garlicvread
garlicvread commented Oct 22, 2022
View reviewed changes
Copy link
Contributor Author

@garlicvread garlicvread left a comment

Choose a reason for hiding this comment

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

이것도 마찬가지로 오후에 한 내용입니다.

garlicvread
garlicvread commented Oct 22, 2022
View reviewed changes
Copy link
Contributor Author

@garlicvread garlicvread left a comment

Choose a reason for hiding this comment

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

전체적으로 오타는 #457 이슈에서 이미 언급된 '떄' 는 이미 수정이 된 상태였고,
그 외에 '커맨드'가 '커멘드'로 적혀 있는 케이스가 하나 있었습니다.

intermediate_source/char_rnn_classification_tutorial.py
`torchtext` 의 편리한 많은 기능들을 사용하지 않고 어떻게 하는지 "기초부터(from scratch)"
보여주기 때문에 NLP 모델링을 위한 전처리가 저수준에서 어떻게 진행되는지를 알 수 있습니다.
단어를 분류하기 위해 기초적인 문자-단위 RNN을 구축하고 학습할 예정입니다.
이 튜토리얼에서는(이후 2개 튜토리얼과 함께) NLP 모델링을 위해 `torchtext` 의
Copy link
Contributor Author

Choose a reason for hiding this comment

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

어색한 번역체를 우리말 문법에 맞게 문장 구조를 수정했습니다.

hyoyoung
hyoyoung approved these changes Dec 2, 2022
View reviewed changes
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.

다음부터는 개별 파일별로, 별도의 PR로 제출 부탁드립니다

good

@hyoyoung hyoyoung merged commit 82e1040 into PyTorchKorea:master Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyoyoung hyoyoung hyoyoung approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@garlicvread @hyoyoung