Skip to content

Conversation

slin1237
Copy link
Collaborator

@slin1237 slin1237 commented Jul 31, 2025

Motivation

  1. GitHub Actions setup-python installs packages in its tool cache path:
    /home/keru/action-runner/_work/_tool/Python/3.11.13/x64/lib/python3.11/site-packages
  2. genai-bench executable is installed in the user's local bin:
    /home/keru/.local/bin/genai-bench
  3. When genai-bench runs, it's looking for its dependencies in:
    /home/keru/.local/lib/python3.11/site-packages

This path mismatch occurs because:

  • The setup-python action creates an isolated Python environment
  • The pip install with --break-system-packages flag installs to the system Python location
  • But the genai-bench script gets installed to the user's local directory
  • When executed, genai-bench uses the Python interpreter's default paths, which don't include the GitHub Actions tool directory

The issue is that genai-bench is installed as a console script entry point, and when it runs, it might not have the same Python path configuration as when it was installed.

Modifications

ci yaml

Accuracy Test

Benchmark & Profiling

Checklist

Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@slin1237 slin1237 merged commit ae80777 into main Jul 31, 2025
15 checks passed
@slin1237 slin1237 deleted the slin/fix branch July 31, 2025 17:40
huangzhilin-hzl pushed a commit to huangzhilin-hzl/sglang that referenced this pull request Aug 1, 2025
TianQiLin666666 pushed a commit to TianQiLin666666/sglang that referenced this pull request Aug 1, 2025
lifuhuang pushed a commit that referenced this pull request Aug 3, 2025
ShangmingCai pushed a commit that referenced this pull request Aug 5, 2025
ShangmingCai pushed a commit that referenced this pull request Aug 5, 2025
narutolhy pushed a commit to narutolhy/sglang that referenced this pull request Aug 17, 2025
narutolhy pushed a commit to narutolhy/sglang that referenced this pull request Aug 18, 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.

1 participant