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

translated gguf.md into chinese #34163

Merged
merged 5 commits into from
Oct 24, 2024

Conversation

blueingman
Copy link
Contributor

What does this PR do?

I am new to the open-source community, but I’m very eager to contribute to the Transformers project. I have translated the gguf.md file into Chinese. If there are any areas that need improvement or if I overlooked anything, please let me know

Before submitting

  • [y] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • [y] 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?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@LysandreJik
Copy link
Member

Nice! Maybe @Isotr0py would you be willing to review this PR? :)

@Isotr0py
Copy link
Contributor

OK, I will take a look tonight. :)

Copy link
Contributor

@Isotr0py Isotr0py 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 contributions! The translation overall LGTM. Just leave some nits to make sentence more coherent.

docs/source/zh/_toctree.yml Outdated Show resolved Hide resolved
docs/source/zh/gguf.md Outdated Show resolved Hide resolved
docs/source/zh/gguf.md Outdated Show resolved Hide resolved
docs/source/zh/gguf.md Outdated Show resolved Hide resolved
docs/source/zh/gguf.md Outdated Show resolved Hide resolved
docs/source/zh/gguf.md Outdated Show resolved Hide resolved
docs/source/zh/gguf.md Outdated Show resolved Hide resolved
I have updated the PR accordingly.Thank you very much for detailed guidance,and I 'll pay more attention to the details next time.

Co-authored-by: Isotr0py <2037008807@qq.com>
@blueingman
Copy link
Contributor Author

I have updated the PR accordingly.Thank you very much for detailed guidance,and I 'll pay more attention to the details next time. @Isotr0py @LysandreJik

Copy link
Contributor

@Isotr0py Isotr0py left a comment

Choose a reason for hiding this comment

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

Just a minor mistake. Otherwise LGTM.

docs/source/zh/_toctree.yml Outdated Show resolved Hide resolved
docs/source/zh/gguf.md Outdated Show resolved Hide resolved
Co-authored-by: Isotr0py <2037008807@qq.com>
@blueingman
Copy link
Contributor Author

Thank you again for your careful suggestion!

@blueingman
Copy link
Contributor Author

Hi @Isotr0py ,hope you're doing well. I have addressed the changes you suggested and updated the PR accordingly. When you have a moment, could you please review the updated code? Thank you for your time!

@Isotr0py
Copy link
Contributor

The updated docs LGTM. :)

Also cc @LysandreJik

Copy link
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@LysandreJik LysandreJik merged commit f0b3ef9 into huggingface:main Oct 24, 2024
7 checks passed
BernardZach pushed a commit to BernardZach/transformers that referenced this pull request Dec 5, 2024
* translated gguf.md into chinese

* Apply suggestions from code review

I have updated the PR accordingly.Thank you very much for detailed guidance,and I 'll pay more attention to the details next time.

Co-authored-by: Isotr0py <2037008807@qq.com>

* Apply suggestions from code review

Co-authored-by: Isotr0py <2037008807@qq.com>

---------

Co-authored-by: Isotr0py <2037008807@qq.com>
BernardZach pushed a commit to innovationcore/transformers that referenced this pull request Dec 6, 2024
* translated gguf.md into chinese

* Apply suggestions from code review

I have updated the PR accordingly.Thank you very much for detailed guidance,and I 'll pay more attention to the details next time.

Co-authored-by: Isotr0py <2037008807@qq.com>

* Apply suggestions from code review

Co-authored-by: Isotr0py <2037008807@qq.com>

---------

Co-authored-by: Isotr0py <2037008807@qq.com>
@blueingman blueingman deleted the translated_into_zh branch December 15, 2024 12:25
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.

3 participants