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

🌐[i18n-KO] Translate autoclass_tutorial to Korean and Fix the typo of quicktour #22533

Merged
merged 1 commit into from
Apr 7, 2023

Conversation

gabrielwithappy
Copy link
Contributor

@gabrielwithappy gabrielwithappy commented Apr 3, 2023

What does this PR do?

Translated the autoclass_tutorial.mdx file of the documentation to Korean and fix the typo of quicktour
Thank you in advance for your review.

Part of #20179

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline,
    Pull Request section?
  • Was this discussed/approved via a Github issue or the forum? Please add a link
    to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the
    documentation guidelines, and
    here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

Team PseudoLab, may you please review this PR?
@0525hhgus, @kihoon71, @sim-so, @gabrielwithappy, @HanNayeoniee, @wonhyeongseo, @jungnerd

May you please review this PR?
@sgugger, @ArthurZucker, @eunseojo

Review result

  • fix a wrong source code link of functions in the document

Links for API documents are not activated. I checked other language documents have same problem. I think it will be fixed when API documents are translated.

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Apr 3, 2023

The documentation is not available anymore as the PR was closed or merged.

Copy link
Collaborator

@sgugger sgugger left a comment

Choose a reason for hiding this comment

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

Thanks for the translation! Let us know when it's good to merge.

@gabrielwithappy
Copy link
Contributor Author

gabrielwithappy commented Apr 3, 2023

  • need to add the english link of Load pretrained instances with an AutoClass
  • keep the AutoClass as a english

Copy link
Contributor Author

@gabrielwithappy gabrielwithappy left a comment

Choose a reason for hiding this comment

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

checked

docs/source/ko/autoclass_tutorial.mdx Outdated Show resolved Hide resolved
@@ -168,7 +168,7 @@ label: NEGATIVE, with score: 0.5309

### AutoTokenizer

토큰나이저는 전처리를 담당하며, 텍스트를 모델이 받을 숫자 배열로 바꿉니다. 토큰화 과정에는 단어를 어디에서 끊을지, 얼만큼 나눌지 등을 포함한 여러 규칙이 있습니다. 자세한 내용은 [토크나이저 요약](./tokenizer_summary)를 확인해주세요. 제일 중요한 점은 모델이 훈련됐을 때와 동일한 토큰화 규칙을 쓰도록 동일한 모델 이름으로 토크나이저 인스턴스를 만들어야 합니다.
토크나이저는 전처리를 담당하며, 텍스트를 모델이 받을 숫자 배열로 바꿉니다. 토큰화 과정에는 단어를 어디에서 끊을지, 얼만큼 나눌지 등을 포함한 여러 규칙이 있습니다. 자세한 내용은 [토크나이저 요약](./tokenizer_summary)를 확인해주세요. 제일 중요한 점은 모델이 훈련됐을 때와 동일한 토큰화 규칙을 쓰도록 동일한 모델 이름으로 토크나이저 인스턴스를 만들어야 합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for catching this sir!

@gabrielwithappy
Copy link
Contributor Author

squashed commit messages and check a final document result.

@gabrielwithappy
Copy link
Contributor Author

@0525hhgus, @kihoon71, @sim-so, @gabrielwithappy, @HanNayeoniee, @wonhyeongseo, @jungnerd
Please review thie PR.
Thank you in advance.

docs/source/ko/autoclass_tutorial.mdx Show resolved Hide resolved
docs/source/ko/autoclass_tutorial.mdx Show resolved Hide resolved
docs/source/ko/autoclass_tutorial.mdx Show resolved Hide resolved

<frameworkcontent>
<pt>
마지막으로 AutoModelFor클래스를 사용하면 주어진 작업에 대해 미리 학습된 모델을 로드할 수 있습니다 (사용 가능한 작업의 전체 목록은 [여기](model_doc/auto)를 참조하세요). 예를 들어, [`AutoModelForSequenceClassification.from_pretrained`]를 사용하여 시퀀스 분류용 모델을 로드할 수 있습니다:
Copy link
Contributor

Choose a reason for hiding this comment

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

마지막으로 AutoModelFor클래스를 사용하면 주어진 작업에 대해 미리 학습된 모델을 로드할 수 있습니다 (사용 가능한 작업의 전체 목록은 여기를 참조하세요). 예를 들어, AutoModelForSequenceClassification.from_pretrained를 사용하여 시퀀스 분류용 모델을 로드할 수 있습니다:

일반적으로 AutoTokenizer 클래스와 AutoModelFor 클래스를 사용하여 미리 학습된 모델 인스턴스를 로드하는 것이 좋습니다. 이렇게 하면 매번 올바른 아키텍처를 로드할 수 있습니다. 다음 [튜토리얼](preprocessing)에서는 새롭게 로드한 토크나이저, 이미지 프로세서, 특징 추출기를 사용하여 미세 튜닝용 데이터 세트를 전처리하는 방법에 대해 알아봅니다.
</pt>
<tf>
마지막으로 `TFAutoModelFor` 클래스를 사용하면 주어진 작업에 대해 사전 훈련된 모델을 로드할 수 있습니다. (사용 가능한 작업의 전체 목록은 [여기](model_doc/auto)를 참조하세요. 예를 들어, [`TFAutoModelForSequenceClassification.from_pretrained`]로 시퀀스 분류를 위한 모델을 로드합니다:
Copy link
Contributor

Choose a reason for hiding this comment

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

마지막으로 TFAutoModelFor 클래스를 사용하면 주어진 작업에 대해 사전 훈련된 모델을 로드할 수 있습니다. (사용 가능한 작업의 전체 목록은 여기를 참조하세요. 예를 들어, TFAutoModelForSequenceClassification.from_pretrained로 시퀀스 분류를 위한 모델을 로드합니다:

@gabrielwithappy
Copy link
Contributor Author

gabrielwithappy commented Apr 5, 2023

[Korean]
음.. 좀 헷갈리네요. 링크가 안되는 이유가 뭔지 좀 알아봐야 될 것 같습니다.
소스코드에 링크가 안걸리는게 문제 같습니다. 좀 더 찾아볼께요
[English]
I will check why the hyperlink does not work.
I think I missed somthing on link code of source codes in the document

@gabrielwithappy
Copy link
Contributor Author

gabrielwithappy commented Apr 6, 2023

Thank you.
I found my source code links are wrong.
I updated review action items and fix it @HanNayeoniee

@gabrielwithappy gabrielwithappy changed the title [WIP]🌐[i18n-KO] Translate autoclass_tutorial to Korean and Fix the typo of quicktour 🌐[i18n-KO] Translate autoclass_tutorial to Korean and Fix the typo of quicktour Apr 6, 2023
@gabrielwithappy
Copy link
Contributor Author

May you please review this PR?
@sgugger, @ArthurZucker, @eunseojo

@sgugger sgugger merged commit d59034f into huggingface:main Apr 7, 2023
@sgugger
Copy link
Collaborator

sgugger commented Apr 7, 2023

Thanks for your contribution!

Copy link
Contributor

@wonhyeongseo wonhyeongseo left a comment

Choose a reason for hiding this comment

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

Congratulations on your first PR merge for HuggingFace, @gabrielwithappy ! 🎉 🎉 🎉
I'm really excited to continue learning/working with you, and hope we can create something fun as well.

P.S. I forgot to post my reviews 😅 May I leave them in a sub-issue as we did with @jungnerd and #22549 ? Most issues are minor, but there is an outstanding issue on L13 we need to resolve.

docs/source/ko/autoclass_tutorial.mdx Show resolved Hide resolved

# AutoClass로 사전 학습된 인스턴스 로드[[Load pretrained instances with an AutoClass]]

트랜스포머 아키텍처가 매우 다양하기 때문에 체크포인트에 맞는 아키텍처를 생성하는 것이 어려울 수 있습니다. 라이브러리를 쉽고 간단하며 유연하게 사용하기 위한 Transformer 핵심 철학의 일환으로, `AutoClass`는 주어진 체크포인트에서 올바른 아키텍처를 자동으로 추론하여 로드합니다. `from_pretrained()` 메서드를 사용하면 모든 아키텍처에 대해 사전 학습된 모델을 빠르게 로드할 수 있으므로 모델을 처음부터 학습하는 데 시간과 리소스를 투입할 필요가 없습니다. 이러한 유형의 체크포인트에 구애받지 않는 코드를 생성한다는 것은 코드가 한 체크포인트에서 작동한다면 아키텍처가 다르더라도 유사한 작업에 대해 학습된 것이라면 다른 체크포인트에서도 작동한다는 것을 의미합니다.
Copy link
Contributor

@wonhyeongseo wonhyeongseo Apr 4, 2023

Choose a reason for hiding this comment

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

마지막 문장이 잘 읽히지 않아요. ~는 것의 표현을 쓰시기 보다 어절 간의 배열을 바꿔보시면 어떨까요?

docs/source/ko/autoclass_tutorial.mdx Show resolved Hide resolved
docs/source/ko/autoclass_tutorial.mdx Show resolved Hide resolved
docs/source/ko/autoclass_tutorial.mdx Show resolved Hide resolved
@gabrielwithappy gabrielwithappy deleted the ko_translation branch April 14, 2023 01:15
novice03 pushed a commit to novice03/transformers that referenced this pull request Jun 23, 2023
…of `quicktour` (huggingface#22533)

translate the autoclass_tutorial and fix the typo of the quicktour
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.

5 participants