-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[sglang, rollout] feat: support sglang as rollout engine in fully async policy #4191
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
wuxibin89
merged 30 commits into
verl-project:main
from
meituan-search:recipe/async_policy_sglang
Jan 19, 2026
Merged
Changes from 7 commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
e18ba0d
sglang rollout in fully async
AniZpZ 0aec584
upd
AniZpZ 48103dc
minor fix
AniZpZ c9e7d09
Merge remote-tracking branch 'origin/main' into recipe/async_policy_s…
AniZpZ 815ebb2
Merge branch 'main' into recipe/async_policy_sglang
AniZpZ 814be85
fix ValueError and MemoryAllocateError
ce64a14
Merge remote-tracking branch 'origin/main' into recipe/async_policy_s…
AniZpZ a1ae8db
fmt
AniZpZ f6c7589
Keep the original logic of enable_memory_saver and reduce the paramet…
0b13a30
Merge remote-tracking branch 'origin/main' into recipe/async_policy_s…
AniZpZ d6e7ab7
update clear_kv_cache
AniZpZ 4492587
upd
AniZpZ 2b0c897
upda
AniZpZ 5c68371
fmt
AniZpZ a3bdc97
Fixed some minor issues with using Megatron to replace FSDP.
8edae0d
Resolved some conflicts and improved parameter synchronization strate…
afb1ad9
Resolved conflict between the partial rollout and the memory saver fe…
f424edb
Fix the issue of incorrect calculation for rollouter/active_time.
8dc3303
Merge remote-tracking branch 'origin/main' into recipe/async_policy_s…
AniZpZ 64aba78
Minor rectification after rebase.
03cc18d
fix pre-commit
jsfanfanfan dbe9a45
minor fix
jsfanfanfan 0121750
minor fix
8257a08
minor fix
5721903
Merge branch 'volcengine:main' into recipe/async_policy_sglang
jsfanfanfan 72ddbb1
fix TypeError: got an unexpected keyword argument 'base_gpu_id'
80f277f
Use lazy import to avoid ModuleNotFoundError
f9994a6
fix base_gpu_id missing!
67096c3
pre-commit
b0f905a
remove reduntant classes and lines.
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| # Copyright 2025 Meituan Ltd. and/or its affiliates | ||
| # | ||
| # Licensed under the Apache License, Version 2.0 (the "License"); | ||
| # you may not use this file except in compliance with the License. | ||
| # You may obtain a copy of the License at | ||
| # | ||
| # http://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. |
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.
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.
Should we unify the use of clear_kv_cache as the interface here? Modifications can be made by rebasing on the main branch.
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.
fixed