Skip to content

Conversation

FeiDaLI
Copy link
Contributor

@FeiDaLI FeiDaLI commented Oct 5, 2025

What type of PR is this?
add script to measure query length impact on response time

What this PR does / why we need it:
Adds a test script that sends HTTP requests with incrementally increasing content lengths to the Semantic Router and measures response times. This helps analyze how query length affects inference performance and latency, and can be integrated with metrics collection systems for deeper performance analysis #338.

Signed-off-by: lillyfuge <[email protected]>
Copy link

github-actions bot commented Oct 5, 2025

👥 vLLM Semantic Team Notification

The following members have been identified for the changed files in this PR and have been automatically assigned:

📁 bench

Owners: @yuezhu1, @Xunzhuo
Files changed:

  • bench/bench_query_length.sh

vLLM

🎉 Thanks for your contributions!

This comment was automatically generated based on the OWNER files in the repository.

Copy link

netlify bot commented Oct 5, 2025

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit 1b27fe3
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/68e285c56073f40008c32423
😎 Deploy Preview https://deploy-preview-350--vllm-semantic-router.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

}

# Create result log file
log_file="test_results_$(date +%Y%m%d_%H%M%S).log"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if saving to the current directory, the file can be added to gitignore

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To avoid saving the file locally, I directly output the results to the shell for easy viewing.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry i forget delete the original code


echo "$content"
}
#!/bin/bash
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a dup

Signed-off-by: lillyfuge <[email protected]>
@FeiDaLI FeiDaLI closed this Oct 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants