Skip to content

Conversation

Judy-Choi
Copy link
Contributor

@Judy-Choi Judy-Choi commented Jul 19, 2025

What does this PR do?

Translated the models.md file of the documentation to Korean.
Thank you in advance for your review.

Part of #20179

Before reviewing

  • Check for missing / redundant translations (번역 누락/중복 검사)
  • Grammar Check (맞춤법 검사)
  • Review or Add new terms to glossary (용어 확인 및 추가)
  • Check Inline TOC (e.g. [[lowercased-header]])
  • Check live-preview for gotchas (live-preview로 정상작동 확인)

Who can review? (Initial)

May you please review this PR?
@4N3MONE, @yijun-lee, @jungnerd , @harheem

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? (Final)

@Rocketknight1
Copy link
Member

cc @stevhliu

Copy link
Member

@stevhliu stevhliu 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 your translation! We can merge once a Korean reader has reviewed as well 🤗

Copy link
Contributor

@4N3MONE 4N3MONE 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

@harheem harheem 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

@jungnerd jungnerd left a comment

Choose a reason for hiding this comment

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

번역하느라 고생 많으셨습니다!
일부 번역에 대해 리뷰를 작성했습니다. 참고 부탁드려요! 🤗

@Judy-Choi Judy-Choi force-pushed the ko-models.md branch 2 times, most recently from ea979d7 to 45a60df Compare July 26, 2025 07:00
> [!TIP]
> *아키텍처*는 모델의 골격을 의미하고 *체크포인트(checkpoint)*는 주어진 아키텍처에 대한 모델의 가중치를 의미합니다. 예를 들어, [BERT](./model_doc/bert)는 아키텍처이고 [google-bert/bert-base-uncased](https://huggingface.co/google-bert/bert-base-uncased)는 해당 아키텍처의 체크포인트(checkpoint)입니다. *모델*이라는 용어는 아키텍처 및 체크포인트(checkpoint)와 혼용하여 사용되는 것을 볼 수 있습니다.

로드할 수 있는 모델에는 두 가지 일반적인 타입이 있습니다:
Copy link
Contributor Author

@Judy-Choi Judy-Choi Jul 26, 2025

Choose a reason for hiding this comment

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

Suggested change
로드할 수 있는 모델에는 두 가지 일반적인 타입이 있습니다:
불러올 수 있는 모델은 일반적으로 두 가지 타입이 있습니다.

Copy link
Contributor Author

@Judy-Choi Judy-Choi left a comment

Choose a reason for hiding this comment

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

추가로 번역 수정했습니다.

print(sorted(os.listdir(tmp_dir)))
```

[`~PreTrainedModel.from_pretrained`]로 샤드된 체크포인트를 다시 로드합니다.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
[`~PreTrainedModel.from_pretrained`]로 샤드된 체크포인트를 다시 로드합니다.
[`~PreTrainedModel.from_pretrained`]로 샤드된 체크포인트를 다시 불러옵니다.

new_model = AutoModel.from_pretrained(tmp_dir)
```

샤드된 체크포인트는 [`~transformers.modeling_utils.load_sharded_checkpoint`]로도 직접 로드할 수 있습니다.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
샤드된 체크포인트는 [`~transformers.modeling_utils.load_sharded_checkpoint`]로도 직접 로드할 수 있습니다.
샤드된 체크포인트는 [`~transformers.modeling_utils.load_sharded_checkpoint`]로도 직접 불러올 수 있습니다.

> [!TIP]
> 장치 할당에 대한 자세한 내용은 [장치 맵 설계하기](https://hf.co/docs/accelerate/v0.33.0/en/concept_guides/big_model_inference#designing-a-device-map)를 참조하세요.

대형 모델 추론의 두 번째 기능은 가중치가 로드되고 모델 스켈레톤에 할당되는 방식과 관련이 있습니다. 모델 가중치는 사용 가능한 모든 디바이스에 분산되며, 가장 빠른 디바이스(보통 GPU)부터 시작해 나머지 가중치는 느린 디바이스(CPU 및 하드 디스크)로 순차적으로 할당됩니다..
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
대형 모델 추론의 두 번째 기능은 가중치가 로드되고 모델 스켈레톤에 할당되는 방식과 관련이 있습니다. 모델 가중치는 사용 가능한 모든 디바이스에 분산되며, 가장 빠른 디바이스(보통 GPU)부터 시작해 나머지 가중치는 느린 디바이스(CPU 및 하드 디스크)로 순차적으로 할당됩니다..
대형 모델 추론의 두 번째 기능은 불러온 가중치가 모델 스켈레톤에 할당되는 방식과 관련이 있습니다. 모델 가중치는 사용 가능한 모든 디바이스에 분산되며, 가장 빠른 디바이스(보통 GPU)부터 시작해 나머지 가중치는 느린 디바이스(CPU 및 하드 디스크)로 순차적으로 할당됩니다..

@Judy-Choi Judy-Choi marked this pull request as ready for review July 29, 2025 12:01
Copy link
Contributor Author

@Judy-Choi Judy-Choi left a comment

Choose a reason for hiding this comment

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

피드백 반영하여 수정을 마쳤습니다. 감사합니다!
@4N3MONE, @jungnerd , @harheem

@jungnerd
Copy link
Contributor

jungnerd commented Aug 1, 2025

@Judy-Choi
안녕하세요 민주님! PR을 살펴보니 리뷰 반영한 커밋이 제대로 적용되지 않은 것 같아요.
혹시 다시 한 번 리뷰 반영해서 커밋해보시겠어요?

@Judy-Choi
Copy link
Contributor Author

Judy-Choi commented Aug 1, 2025

@Judy-Choi 안녕하세요 민주님! PR을 살펴보니 리뷰 반영한 커밋이 제대로 적용되지 않은 것 같아요. 혹시 다시 한 번 리뷰 반영해서 커밋해보시겠어요?

@jungnerd 안녕하세요 멘토님! 다시 한 번 커밋했습니다. 확인해 주시면 감사하겠습니다!

@harheem
Copy link
Contributor

harheem commented Aug 1, 2025

@Judy-Choi 민주님 안녕하세요! 반영해주신 것 확인했습니다. 감사합니다 ㅎㅎ
glossary를 무척 잘 반영해주셨는데, 혹시 사전훈련된 모델 -> 사전 훈련된 모델로 변경하는 것에 대해서 어떻게 생각하시나요?

Copy link
Contributor Author

@Judy-Choi Judy-Choi left a comment

Choose a reason for hiding this comment

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

@harheem 꼼꼼하게 확인해 주셔서 감사합니다! ㅎㅎ
멘토님께서 제안해 주신 대로 사전훈련된 -> 사전 훈련된 으로 일괄적으로 띄어쓰기 적용하니 더 가독성이 좋아졌습니다. 감사합니다!

Copy link
Contributor

@harheem harheem left a comment

Choose a reason for hiding this comment

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

정말 고생많으셨습니다 민주님 그리고 감사합니다 🤗
늦은 리뷰 죄송하며, merge가 눈앞이네요!

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Member

@stevhliu stevhliu left a comment

Choose a reason for hiding this comment

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

Thanks! Remember to also update the toctree with the translation!

Copy link
Member

@stevhliu stevhliu left a comment

Choose a reason for hiding this comment

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

local should reflect the file name :)

@stevhliu
Copy link
Member

build-doc

Copy link
Contributor

@stevhliu stevhliu merged commit c81723d into huggingface:main Aug 25, 2025
15 checks passed
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.

7 participants