-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Support SGLang as Potential Backend for Evaluation #2703
Open
Monstertail
wants to merge
18
commits into
EleutherAI:main
Choose a base branch
from
Monstertail:sglang-support
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
8ebbaf0
initial components to support sglang
Monstertail d458f64
init of class SGLangLM
Monstertail 5444319
draft for generate_until of SGLang model
Monstertail 13483f0
mock loglikelihood
Monstertail 5d0dd5d
initial loglikelihood_tokens
Monstertail 59b890c
todo: fix bug of sglang engine init
Monstertail 97b639f
implement generation tasks and test
Monstertail 605906d
support output type loglikelihood and loglikelihood_rolling (#1)
XiaotongJiang 0d7a1b4
support dp_size>1
Monstertail 2c1d0d3
typo
Monstertail 3fafe7c
add tests and clean code
Monstertail 64f2964
skip tests of sglang for now
Monstertail 5e8073b
fix OOM error of sglang pytest
Monstertail b433138
finish test for sglang
Monstertail 19007d7
add sglang to readme
Monstertail 96d4d5b
fix OOM of tests and clean SGLang model
Monstertail 4513b00
update readme
Monstertail bb6e0df
clean pyproject and add tests for evaluator
Monstertail 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 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
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.
Remove this. Just saying:
To use SGLang as evaluation backend, please install it in advance(due to dependencies of
Flashinfer
, a fast attention kernel library). See docs here to install SGLang.Delete the
uv
commands, since it maybe updated in SGLang.