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

Translated korean_comments.md (#139) #143

Merged
merged 2 commits into from
Oct 25, 2020
Merged

Translated korean_comments.md (#139) #143

merged 2 commits into from
Oct 25, 2020

Conversation

warnikchow
Copy link
Contributor

@warnikchow warnikchow commented Oct 25, 2020

1. 해당 PR은 어떤 내용인가요?

#142 에 이어 korean_comments를 번역하였습니다. #141 을 많이 참고하였으며, 구조적인 부분은 앞으로도 비슷한 방식으로 진행될 것 같습니다.

2. PR과 관련된 이슈가 있나요?

  • Korean news comments data가 직역이긴 하지만, KcBERT Pre-Training Corpus 라는 캐글 배포 이름을 사용하였습니다.
  • ko-docs 원문에 text(질문) 으로 되어 있는 부분을 text(뉴스 댓글)로 수정해야 할 것 같습니다. 이 부분은 별도의 PR를 드리도록 하겠습니다.

@warnikchow warnikchow requested a review from lovit October 25, 2020 08:03
Copy link
Member

@lovit lovit left a comment

Choose a reason for hiding this comment

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

번역 내용 확인하였습니다.

@lovit
Copy link
Member

lovit commented Oct 25, 2020

ko-docs 원문에 text(질문) 으로 되어 있는 부분을 text(뉴스 댓글)로 수정해야 할 것 같습니다. 이 부분은 별도의 PR를 드리도록 하겠습니다.

위 내용이 어떤 내용을 말씀하시는지 찾지 못하였습니다. 이러한 내용은 논의 없이 곧바로 issues 에 올려주시면 됩니다!

@warnikchow warnikchow merged commit 72a1ad6 into pages Oct 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants