Skip to content

Conversation

@kwonmha
Copy link
Contributor

@kwonmha kwonmha commented Sep 11, 2020

라이선스 동의

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

관련 이슈 번호

PR 종류

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

PR 설명

beginner_source/nlp/advanced_tutorial.py 번역

@codingbowoo
Copy link
Member

컨트리뷰톤 리뷰하겠습니다 :)

Copy link
Contributor

@Gangsss Gangsss left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Contributor

@potatochips178 potatochips178 left a comment

Choose a reason for hiding this comment

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

번역하시느라 고생하셨습니다. 워낙 깔끔하게 하셔서 수정할 곳이 거의 없네요,,,

정의합니다. :math:`i`번째 단어에 대한 배출 포텐셜은 Mi-LSTM의
:math:`i`번째 시점의 은닉 상태가 결정합니다. 전이 점수는 :math:`|T|x|T|`
형태인 행렬 :math:`\textbf{P}`에 저장되어 있습니다. :math:`T`는
태그의 집합입니다. 이 구현에서, :math:`\textbf{P}_{j,k}`는 tag :math:`j`에서
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
태그의 집합입니다. 구현에서, :math:`\textbf{P}_{j,k}` tag :math:`j`에서
태그의 집합으로 :math:`\textbf{P}` 위치합니다. 구현에서, :math:`\textbf{P}_{j,k}` tag :math:`j`에서

원문의 뜻을 최대한 그대로 담기위해 번역을 조금 수정했습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

제안해주신대로 수정하면
"T 는 태그의 집합으로 P에 위치합니다" 라는 문장이 되는 것 같습니다.

"P는 |T| X |T| 형태의 전이점수 행렬입니다.
T가 태그의 집합입니다."
제가 문장을 두 개로 나눴는데 확인 부탁드립니다.

Copy link
Member

@codingbowoo codingbowoo left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 뒷부분은 앞의 두 분께서 봐주셔서 앞의 몇 문단에 해당하는 리뷰만 남깁니다.

Co-authored-by: 장보우 Bowoo Jang <lisabowoo@gmail.com>
Co-authored-by: potatochips178 <57126990+potatochips178@users.noreply.github.com>
Co-authored-by: Gangmin-Kim <42054789+Gangsss@users.noreply.github.com>
@kwonmha
Copy link
Contributor Author

kwonmha commented Sep 12, 2020

리뷰 감사드립니다!

@9bow 9bow added the 컨트리뷰톤 오픈소스 컨트리뷰톤 관련 이슈/PR label Sep 13, 2020
@creduo creduo linked an issue Sep 14, 2020 that may be closed by this pull request
@9bow 9bow requested review from 9bow, adonisues, bongmo and creduo November 5, 2020 13:54
Copy link
Member

@9bow 9bow left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다!
reST 문법 관련하여 수정 사항이 다소 있지만, 리뷰가 늦어진 점을 감안하여 #175 에 포함하여 제가 수정하도록 하겠습니다.
감사합니다!

@9bow 9bow merged commit 46cadbb into PyTorchKorea:master Nov 8, 2020
@9bow 9bow added the 추가작업필요 병합 후 추가 수정이 필요한 경우 label Nov 22, 2020
@kwonmha kwonmha deleted the nlp-advanced_tutorial branch December 28, 2020 07:24
@9bow 9bow removed the 추가작업필요 병합 후 추가 수정이 필요한 경우 label Apr 11, 2021
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.

beginner_source/nlp/advanced_tutorial.py 번역

5 participants