-
Notifications
You must be signed in to change notification settings - Fork 47
Improve documentation for CPU-only users and first-time contributors #474
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Improve documentation for CPU-only users and first-time contributors
Summary of ChangesHello @P09s, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly improves the project's documentation, making it more accessible and user-friendly for a broader audience. It specifically addresses the needs of CPU-only users by providing a streamlined quick-start guide and clarifies hardware requirements. Additionally, the contribution section has been refined to encourage and guide first-time contributors, lowering the barrier to entry for community involvement. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
Hi maintainers, this PR improves documentation for CPU-only users and first-time contributors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request significantly improves the onboarding experience for new users by adding a CPU-only quick start guide, clarifying hardware requirements, and making the contribution section more welcoming. The changes are well-structured and greatly enhance the clarity of the README.md file. My review includes a couple of suggestions to address minor inconsistencies in the installation instructions, which will further improve the documentation's consistency and help prevent potential setup issues for users.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
pan-x-c
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your suggestion.
Co-authored-by: Xuchen Pan <32844285+pan-x-c@users.noreply.github.com>
Co-authored-by: Xuchen Pan <32844285+pan-x-c@users.noreply.github.com>
Updated installation section to improve clarity and added GPU and CPU-only support details.
Removed unnecessary line breaks in the README for better readability.
|
Thank you for the feedback. I’ve separated the GPU requirements and CPU-only support as discussed and cleaned up the README formatting that was causing the pre-commit check to fail. The latest commits should now pass CI. |
yanxi-chen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see the inline comments. Some deleted statements are actually useful or more precise, and I think it would be good to bring them back. Otherwise lgtm.
| > **No GPU? No problem!** You can still try it out using the Tinker backend: | ||
| > 1. Follow the installation steps (skip GPU-specific packages like `flash-attn`) | ||
| > 2. Run the **[Tinker training example](https://github.com/modelscope/Trinity-RFT/tree/main/examples/tinker)**, which is designed for CPU-only systems. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These lines can be removed.
| #### CPU-Only Support | ||
|
|
||
| - CPU-only execution is supported via the Tinker backend. | ||
| - This mode is intended for testing, development, and experimentation. | ||
| (see: Minimal CPU-Only Quick Start) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These lines can be removed.
| trinity studio --port 8080 | ||
| ``` | ||
|
|
||
| Then you can configure your RFT process in the web page and generate a config file. You can save the config file for later use or run it directly as described in the following section. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep this line
| For example, to log in to Wandb: | ||
|
|
||
| ```shell | ||
| export WANDB_API_KEY=<your_api_key> | ||
| wandb login | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep these lines
| trinity run --config examples/grpo_gsm8k/gsm8k.yaml | ||
| ``` | ||
|
|
||
| For studio users, click "Run" in the web interface. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep this line
| + we have also drawn inspirations from RL frameworks like [OpenRLHF](https://github.com/OpenRLHF/OpenRLHF), [TRL](https://github.com/huggingface/trl), [ChatLearn](https://github.com/alibaba/ChatLearn) and [rLLM](https://github.com/rllm-org/rllm); | ||
| + ...... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert to the original version
Co-authored-by: Yanxi Chen <153061753+yanxi-chen@users.noreply.github.com>
Co-authored-by: Yanxi Chen <153061753+yanxi-chen@users.noreply.github.com>
Co-authored-by: Yanxi Chen <153061753+yanxi-chen@users.noreply.github.com>
Co-authored-by: Yanxi Chen <153061753+yanxi-chen@users.noreply.github.com>
Co-authored-by: Yanxi Chen <153061753+yanxi-chen@users.noreply.github.com>
Co-authored-by: Yanxi Chen <153061753+yanxi-chen@users.noreply.github.com>
This PR improves onboarding for new users by:
These changes help lower the entry barrier for first-time users.