Skip to content

Conversation

mreraser
Copy link
Contributor

What does this PR do?

Translated the fine_tuning_detr_custom_dataset.ipynb file to Korean.

Before reviewing

  • Check for missing / redundant translations (번역 누락/중복 검사)
  • Grammar Check (맞춤법 검사)
  • Review or Add new terms to glossary (용어 확인 및 추가)

Who can review?

Hello @merveenoyan ! May you please review this PR? Thank you!
Have a nice day :)

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@merveenoyan
Copy link
Collaborator

Thanks a ton! Can we get another Korean reviewer to check this? 👀

Copy link

review-notebook-app bot commented May 27, 2025

View / edit / reply to this conversation on ReviewNB

jungnerd commented on 2025-05-27T08:22:21Z
----------------------------------------------------------------

라벨도 맞는 표현이지만 인공지능 분야에서는 레이블이라고 표현을 더 많이 하는 것 같아, 라벨 -> 레이블 제안드립니다!


Copy link

review-notebook-app bot commented May 27, 2025

View / edit / reply to this conversation on ReviewNB

jungnerd commented on 2025-05-27T08:22:22Z
----------------------------------------------------------------

이미지를 불러오는데 오류가 생기니 이미지를 제거하는 것을 제안드립니다!


@jungnerd
Copy link
Contributor

I reviewed this PR. Please check @mreraser 👍

@mreraser
Copy link
Contributor Author

Thank you @jungnerd for your time and support! 👍

Hello @merveenoyan, I have fixed some translations.
May I kindly ask if you could review this PR one more time? 😄

@mreraser
Copy link
Contributor Author

Hello @merveenoyan , I hope you're having a great day 😄.

Could you please take another look at the PR when you have a chance? Your review would be very helpful. Thank you for your time!

@mreraser
Copy link
Contributor Author

Hello @stevhliu 😄
I hope you're having a great day. Could you please take another look at the PR when you have a chance? Your review would be very helpful. Thank you for your time!

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.

Sorry for the delay, LGTM! Would you mind resolving the conflict and then we can merge?

@mreraser
Copy link
Contributor Author

Sorry for the delay, LGTM! Would you mind resolving the conflict and then we can merge?

That's okay! I've resolved the conflicts. 🚀
Thanks a lot, as always.

@stevhliu stevhliu merged commit 9318e2e into huggingface:main Sep 16, 2025
1 check passed
@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.

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