Skip to content
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

Add vllm benchmark #10600

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add vllm benchmark #10600

wants to merge 1 commit into from

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Jul 1, 2024

Add vllm benchmark

Hi,

I was playing around with a vLLM benchmark and wanted to get your opinion on how to best deal with large files in benchmarks. The vLLM benchmark serves a model (facebook/opt-125m) and uses vLLM's own benchmark tooling to query it with a widely-used dataset (..). For development I just mounted the model and dataset into the container-under-test. But that seems impractical for CI/other devs.

Should I embed the model files into the benchmark image? Is there another way you would prefer?

Please don't do an actual review yet. Just wanted to get some early feedback. I still have to understand metricsviz and how to incorporate it.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#10512 from derpsteb:vllm-benchmark 1b9f8f3

@copybara-service copybara-service bot added the exported Issue was exported automatically label Jul 1, 2024
@copybara-service copybara-service bot force-pushed the test/cl648472759 branch 2 times, most recently from 59aaeae to 6362a8a Compare July 3, 2024 19:52
Hi,

I was playing around with a vLLM benchmark and wanted to get your opinion on how to best deal with large files in benchmarks. The vLLM benchmark serves a model (facebook/opt-125m) and uses vLLM's own benchmark tooling to query it with a widely-used dataset (..). For development I just mounted the model and dataset into the container-under-test. But that seems impractical for CI/other devs.

Should I embed the model files into the benchmark image? Is there another way you would prefer?

Please don't do an actual review yet. Just wanted to get some early feedback. I still have to understand `metricsviz` and how to incorporate it.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#10512 from derpsteb:vllm-benchmark 1b9f8f3
PiperOrigin-RevId: 648472759
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exported Issue was exported automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant