-
Notifications
You must be signed in to change notification settings - Fork 563
feat(sc): support PPO in single controller #3773
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
Merged
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
5b7f9dd
feat(sc): add the driver-side TQValue for the PPO critic
yuki-97 b5ba0eb
feat(sc): add the worker-side TQ value forward for the PPO critic, wi…
yuki-97 f7e90f3
refactor: move the PPO estimator config to a BaseModel, pin estimator…
yuki-97 d597342
feat(sc): add the ppo algorithm block to the SingleController config
yuki-97 ec4ac13
feat(sc): build the PPO value model in SingleController setup, valida…
yuki-97 f60ec32
refactor: name the SingleController test modules after what they test
yuki-97 a638ccc
feat(sc): accept PPO runs in the SingleController entrypoint, make pp…
yuki-97 55c73ac
feat(sc): size the train cluster for the critic, rename the GAE logpr…
yuki-97 596a4bd
feat(sc): run the PPO critic stages in the SingleController train pump
yuki-97 74738e5
feat(sc): skip the weight sync during PPO critic warmup, require the …
yuki-97 8edca3d
feat(sc): run ppo.ppo_epochs optimizer steps per RL step, fix the sta…
yuki-97 eab2fbe
feat(sc): widen the sampler lookahead during PPO critic warmup, port …
yuki-97 7e9d9a4
feat(sc): widen the sampler lookahead during PPO critic warmup, port …
yuki-97 a63dcc9
test(sc): add an async PPO SingleController functional test and its M…
yuki-97 42f7062
test(sc): add an async PPO SingleController nightly recipe
yuki-97 e95ce03
pyrefly
yuki-97 59b2845
test(sc): regroup the SingleController L1 functional test list, raise…
yuki-97 b9bd64c
fix(sc): mark returns/values token-aligned, migrate the data_plane co…
yuki-97 4025501
fix(sc): reject the shaping knobs the SingleController path does not …
yuki-97 0ef34c8
docs(sc): document PPO on the SingleController path, correct the stal…
yuki-97 aa06643
fix(sc): re-enable the DDP forward pre-hook in finish_train_step
yuki-97 0b59fe4
chore(sc): restore the L1 functional test rationale, put the SC-utils…
yuki-97 c253574
fix(sc): require a Megatron critic and reject drop budgets under PPO,…
yuki-97 018cc7e
test(sc): remove four duplicated PPO unit tests, parametrize five nea…
yuki-97 a091a6f
test(sc): set the first-train-step pre-hook state on the fabricated w…
yuki-97 842adb9
docs(sc): document the PPO value stages and critic warmup, note the P…
yuki-97 91c7cc9
test(sc): cover the PPO warmup-lookahead, negative-KL, and gate-windo…
yuki-97 ba8357a
fix(sc): reject colocated generation on the SingleController path
yuki-97 7d1f222
fix(sc): move the exactly-one-algorithm-block check onto MasterConfig…
yuki-97 6d303d3
fix(sc): park the policy before the critic stages when neither logpro…
yuki-97 b60aee0
fix unit test
yuki-97 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.