From 975009cf32c54b8ec62021b0be812d27c9e4d0d6 Mon Sep 17 00:00:00 2001 From: Jiyue Wang Date: Thu, 11 Sep 2025 12:32:25 -0700 Subject: [PATCH 1/5] Some minor cleanup to README.md Summary: title Test Plan: Reviewers: Subscribers: Tasks: Tags: --- README.md | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/README.md b/README.md index 82db7519c..e329db1c0 100644 --- a/README.md +++ b/README.md @@ -22,9 +22,9 @@ conda activate forge Optional: By default, the packages installation uses conda. If user wants to install system packages on the target machine instead of conda, they can pass the `--use-sudo` to the installation script: `./script/install.sh --use-sudo`. -After install, you can run the following command and should see output confirming GRPO training is running. +After install, you can run the following command and should see output confirming GRPO training is running (you need a minimum 3 GPU devices). ``` -python -m apps.grpo.main +python -m apps.grpo.main --config apps/grpo/qwen3_1_7b.yaml ``` If you need to re-build the wheels for whatever reason, you can do so with: @@ -32,12 +32,8 @@ If you need to re-build the wheels for whatever reason, you can do so with: ./scripts/build_wheels.sh ``` -Since the vLLM wheel is too large for GitHub, we uploaded it as a release: -``` -$ gh release create v0.0.0 assets/wheels/vllm-*.whl --title "Forge Wheels v0.0.0" -``` +### Meta Internal Build (Alternative Route) -### Meta Internal Build 1. Build uv package From 77f17cfb880080f129de5e6332eaf07d6d1f2ae5 Mon Sep 17 00:00:00 2001 From: Jiyue Wang Date: Thu, 11 Sep 2025 12:32:25 -0700 Subject: [PATCH 2/5] Some minor cleanup to README.md Summary: title Test Plan: Reviewers: Subscribers: Tasks: Tags: --- README.md | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index e329db1c0..1112f0e47 100644 --- a/README.md +++ b/README.md @@ -22,9 +22,9 @@ conda activate forge Optional: By default, the packages installation uses conda. If user wants to install system packages on the target machine instead of conda, they can pass the `--use-sudo` to the installation script: `./script/install.sh --use-sudo`. -After install, you can run the following command and should see output confirming GRPO training is running (you need a minimum 3 GPU devices). +After install, you can run the following command and should see output confirming GRPO training is running (you need a minimum 3 GPU devices) (you need a minimum 3 GPU devices). ``` -python -m apps.grpo.main --config apps/grpo/qwen3_1_7b.yaml +python -m apps.grpo.main --config apps/grpo/qwen3_1_7b.yaml --config apps/grpo/qwen3_1_7b.yaml ``` If you need to re-build the wheels for whatever reason, you can do so with: @@ -32,8 +32,11 @@ If you need to re-build the wheels for whatever reason, you can do so with: ./scripts/build_wheels.sh ``` -### Meta Internal Build (Alternative Route) +For your information, since the vLLM wheel is too large for GitHub, we uploaded it as a release in the `install.sh` script: +``` +$ gh release create v0.0.0 assets/wheels/vllm-*.whl --title "Forge Wheels v0.0.0" +### Meta Internal Build (Alternative Route) 1. Build uv package From 616bb03d7b9e2e16d63e9a1295f8cdb20b8bf5b1 Mon Sep 17 00:00:00 2001 From: Jiyue Wang Date: Fri, 12 Sep 2025 10:33:20 -0700 Subject: [PATCH 3/5] [EZ] fix broken markdown in README.md Summary: title Test Plan: Reviewers: Subscribers: Tasks: Tags: --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index 1112f0e47..8025baaaa 100644 --- a/README.md +++ b/README.md @@ -35,6 +35,7 @@ If you need to re-build the wheels for whatever reason, you can do so with: For your information, since the vLLM wheel is too large for GitHub, we uploaded it as a release in the `install.sh` script: ``` $ gh release create v0.0.0 assets/wheels/vllm-*.whl --title "Forge Wheels v0.0.0" +``` ### Meta Internal Build (Alternative Route) From abd350cc58a1334baef9ecd41653f9aeba2ae98d Mon Sep 17 00:00:00 2001 From: Jiyue Wang Date: Fri, 12 Sep 2025 10:55:31 -0700 Subject: [PATCH 4/5] Summary: more fixes Test Plan: Reviewers: Subscribers: Tasks: Tags: --- README.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index 8025baaaa..94c7815a0 100644 --- a/README.md +++ b/README.md @@ -22,7 +22,8 @@ conda activate forge Optional: By default, the packages installation uses conda. If user wants to install system packages on the target machine instead of conda, they can pass the `--use-sudo` to the installation script: `./script/install.sh --use-sudo`. -After install, you can run the following command and should see output confirming GRPO training is running (you need a minimum 3 GPU devices) (you need a minimum 3 GPU devices). +After install, you can run the following command and should see output confirming GRPO training is running (you need a minimum 3 GPU devices). + ``` python -m apps.grpo.main --config apps/grpo/qwen3_1_7b.yaml --config apps/grpo/qwen3_1_7b.yaml ``` From ca0c5d588559220df4601b4651951cb81475686b Mon Sep 17 00:00:00 2001 From: Jiyue Wang Date: Fri, 12 Sep 2025 11:11:20 -0700 Subject: [PATCH 5/5] remove double config --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 94c7815a0..aed0632c1 100644 --- a/README.md +++ b/README.md @@ -25,7 +25,7 @@ Optional: By default, the packages installation uses conda. If user wants to ins After install, you can run the following command and should see output confirming GRPO training is running (you need a minimum 3 GPU devices). ``` -python -m apps.grpo.main --config apps/grpo/qwen3_1_7b.yaml --config apps/grpo/qwen3_1_7b.yaml +python -m apps.grpo.main --config apps/grpo/qwen3_1_7b.yaml ``` If you need to re-build the wheels for whatever reason, you can do so with: