Skip to content

Conversation

Kim-Ju-won
Copy link
Contributor

@Kim-Ju-won Kim-Ju-won commented Oct 14, 2025

What does this PR do?

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

Part of #1607

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?
@AhnJoonSung, @nsbg, @yijun-lee, @FacerAin, @Judy-Choi, @HyunZ118, @jungnerd

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)

class HFModelDownloadsTool(Tool):
name = "model_download_counter"
description = """
이 도구는 Hugging Face Hub에서 주어진 태스크에 대해 가장 많이 다운로드된 모델을 반환합니다.

Choose a reason for hiding this comment

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

현재 번역:
이 도구는 Hugging Face Hub에서 주어진 태스크에 대해 가장 많이 다운로드된 모델을 반환합니다. 체크포인트(checkpoint)의 이름을 반환합니다.

원문:
This is a tool that returns the most downloaded model of a given task on the Hugging Face Hub. It returns the name of the checkpoint.

사소한 부분이지만 현재 번역에선 "반환합니다"가 두번 반복되어 의미 파악이 어려운 것 같습니다.

다음과 같이 제안 드리오니 느낌만 참고해주시면 감사하겠습니다.

제안: 이 도구는 주어진 태스크에 대해 Hugging Face Hub에서 가장 많이 다운로드된 모델의 체크포인트(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.

@AhnJoonSung 안녕하세요! 빠르게 리뷰해주셔서 감사합니다. 말씀해주신 해석이 맞습니다만, 해당 문구는 시스템 프롬프트에 그대로 삽입되는 부분이라 의역보다는 직역으로 처리했습니다. 모델이 이 문장을 그대로 해석하기 때문에, 의미가 약간 반복되더라도 원문 구조를 유지하는 것이 Agent나 LLM의 출력이 유지된다고 생각해서요!
다만 ‘체크포인트’ 부분만 내부 glossary을 참고해 반영했습니다.

Choose a reason for hiding this comment

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

아하 그렇군요 주원님 의견이 타당해 보입니다!

- 도구 자체의 이름에 해당하는 `name` 속성입니다. 이름은 보통 도구가 하는 일을 설명합니다. 이 코드는 특정 작업에서 가장 많이 다운로드된 모델을 반환하므로 `model_download_counter`라고 이름 짓겠습니다.
- 에이전트의 시스템 프롬프트를 채우는 데 사용되는 `description` 속성입니다.
- `inputs` 속성은 `"type"`과 `"description"` 키를 갖는 딕셔너리로, 입력값에 대한 정보를 제공합니다. 이를 통해 파이썬 인터프리터가 입력 정보에 맞게 더 잘 처리할 수 있습니다.
- 출력 유형을 지정하는 `output_type` 속성입니다. `inputs`와 `output_type`의 유형은 모두 [Pydantic 형식](https://docs.pydantic.dev/latest/concepts/json_schema/#generating-json-schema)이어야 하며, [`~AUTHORIZED_TYPES`] 중 하나일 수 있습니다.

Choose a reason for hiding this comment

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

원문에서의 예시와 달라진 부분이 있는데 다른 의도가 있는것일까요?

원문: they can be either of these: ["string", "boolean","integer", "number", "image", "audio", "array", "object", "any", "null"].

현재 번역: [~AUTHORIZED_TYPES] 중 하나일 수 있습니다.

Copy link
Contributor Author

@Kim-Ju-won Kim-Ju-won Oct 15, 2025

Choose a reason for hiding this comment

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

@AhnJoonSung 체크해주셔서 감사합니다! 제가 참고한 원본은 이곳(1.21.1)인데, 문서가 계속 수정되고 있어서 제가 main을 Pull 받은 시점에서 코드 수정이 이루어진 것 같습니다~ 해당 부분 반영해서 수정했습니다! 확인해주셔서 감사합니다 👍

Screenshot 2025-10-15 at 12 30 25 PM

@Kim-Ju-won
Copy link
Contributor Author

@AhnJoonSung 빠른 리뷰 너무 감사드려요 :) 수정한 부분 한 곳 반영했고, 한 곳은 생각해볼 부분이라 의견을 남겨두었습니다! 확인 부탁드립니다 👍 섬세하게 살펴봐주셔서 감사드려요!

@AhnJoonSung
Copy link

@AhnJoonSung 빠른 리뷰 너무 감사드려요 :) 수정한 부분 한 곳 반영했고, 한 곳은 생각해볼 부분이라 의견을 남겨두었습니다! 확인 부탁드립니다 👍 섬세하게 살펴봐주셔서 감사드려요!

확인했습니다~ 수고하셨습니다!

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