Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

translation: Data Analyst agent into Korean#212

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
ahnjj wants to merge4 commits intohuggingface:main
base:main
Choose a base branch
Loading
fromahnjj:ko-agent_data_analyst.ipynb

Conversation

ahnjj
Copy link

What does this PR do?

this is PR presents the korean translation ofData analyst agent: get your data’s insights in the blink of an eye ✨ cookbook.

Who can review?

@jeongiin,@harheem,@boyunJang,@ahnjj,@SeungAhSon,@jun048098,@win2dvp21

@review-notebook-appReview Notebook App
Copy link

Check out this pull request on ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered byReviewNB

@ahnjjahnjj marked this pull request as ready for reviewOctober 5, 2024 08:26
@merveenoyan
Copy link
Collaborator

@ahnjj 고맙습니다 once it's reviewed by another Korean speaker we can merge 🫰🏻

@jun048098
Copy link
Contributor

LGTM! 번역 감사합니다!

ahnjj reacted with thumbs up emoji

@merveenoyan
Copy link
Collaborator

@ahnjj we just merged a Korean recipe, toctree and index md has a merge conflict. Can you resolve it? We can merge afterwards.

ahnjj reacted with thumbs up emoji

@merveenoyan
Copy link
Collaborator

Sorry, another Korean recipe was merged, can you resolve again and we can merge this one
Also feel free to send a join request to thisorg.

@ahnjj
Copy link
Author

Sorry, another Korean recipe was merged, can you resolve again and we can merge this one Also feel free to send a join request to thisorg.

Thank you for your help!
Since I've just resolved this, can you help with me to be merged?

Copy link
Contributor

@jun048098jun048098 left a comment

Choose a reason for hiding this comment

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

LGTM approve change

@merveenoyan
Copy link
Collaborator

@ahnjj if you can resolve the merge conflict we can merge
also don't forget to join this org:https://huggingface.co/huggingcooks

@ahnjjahnjj closed thisMar 19, 2025
@ahnjjahnjjforce-pushed theko-agent_data_analyst.ipynb branch fromd8dfa67 to9cd571bCompareMarch 19, 2025 23:44
@ahnjjahnjj reopened thisMar 19, 2025
@ahnjj
Copy link
Author

@merveenoyan
Hi, sorry for the late modification!
I have resolved the conflict issue. If you could review this again and help with the merge, I would really appreciate it.

@ahnjjahnjj requested a review fromjun048098March 21, 2025 00:25
@ahnjj
Copy link
Author

ahnjj commentedApr 1, 2025•
edited
Loading

Hey@merveenoyan , I’d love to get this translation merged.
This PR has been reviewed and is now awaiting review & approval with write access. Could you take a look when you have time? Let me know if any further changes are needed. Appreciate with your time!

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@jun048098jun048098Awaiting requested review from jun048098

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@ahnjj@merveenoyan@jun048098
[8]ページ先頭
©2009-2025 Movatter.jp