-
Notifications
You must be signed in to change notification settings - Fork 210
feat: Add SGLang rollout backend and tests #1674
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
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
…a server Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
…p servers Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
sglang: add 1B example Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Ryan <[email protected]> Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Zhuoran Yin <[email protected]>
- Convert SGLangConfig from regular class to TypedDict inheriting GenerationConfig - Align structure with VllmConfig pattern for consistency - Mark all fields as NotRequired for backward compatibility - Add sglang_kwargs field for additional ServerArgs parameters - Add type casting in grpo.py for type safety This maintains backward compatibility while aligning with the existing generation config structure pattern. Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Zhuoran Yin <[email protected]>
Signed-off-by: Zhuoran Yin <[email protected]>
Co-authored-by: Terry Kong <[email protected]> Signed-off-by: Night <[email protected]>
Signed-off-by: RolaoDenthu <[email protected]>
fixed the lint and the file name. |
Signed-off-by: RolaoDenthu <[email protected]>
|
quote lint check
|
|
quote build-container job
|
Signed-off-by: RolaoDenthu <[email protected]>
Signed-off-by: RolaoDenthu <[email protected]>
I have updated the uv.lock and pyrefly.toml. |
Signed-off-by: RolaoDenthu <[email protected]>
|
@RolaoDenthu lint seems to be complaining a new file not added to the pyrefly.toml |
|
Signed-off-by: RolaoDenthu <[email protected]>
Signed-off-by: RolaoDenthu <[email protected]>
|
@guyueh1 Hi I’ve fixed the environment and it should now be able to run with the current uv lock. Could you please restart the test? |
|
What does this PR do ?
Add comprehensive test coverage for SGLang generation backend, including functional tests, unit tests, and nightly tests.
Usage
Before your PR is "Ready for review"
Pre checks:
Summary by CodeRabbit
Release Notes
New Features
Configuration
Tests
✏️ Tip: You can customize this high-level summary in your review settings.