agentic client: venv rebuilt on every run and caller's UV_CACHE_DIR overridden
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 58/100
Línea de trabajo
Start with infx/bench/agentic/venv.py and the existing expectations in infx/tests/bench/test_agentic_runtime.py:70-97; infx/bench/agentic/run.py:248-256 shows where bootstrap is called. Read the venv setup and tests first, then update the tests for cache selection and matching-venv reuse. Done when matching requirements avoid uv calls, changed requirements rebuild, and a caller-supplied UV_CACHE_DIR is honored.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The agentic benchmark client rebuilds its venv on every run and ignores a caller-supplied UV_CACHE_DIR, so a caller that runs many benchmarks pays a full client download each time. Downstream: AMD-AGI/Magpie#128, AMD-AGI/Hyperloom#1919.
Current behaviour (main @ 288471b, read from code)
inferencex-e2e/benchmarks/srt_agentic.shrunspython3 -m infx.bench agentic, which callsvenv.bootstrapwhen not already inside the client venv (infx/bench/agentic/run.py:248-256).bootstrapalways deletes the venv (infx/bench/agentic/venv.py:66), with no check of whether the existing one already matches.- It always sets
UV_CACHE_DIR=<runtime root>/uv-cache(venv.py:67-69), overriding a value from the caller. The CI workflows set their ownUV_CACHE_DIR(e.g..github/workflows/benchmark-tmpl.yml:246), which this also overrides. - The runtime root is
AIPERF_RUNTIME_DIR, else a per-job temp directory (venv.py:28-34). When the root is new per run, as it is for Magpie's AgentX runs, the cache is always cold and every run downloads the full client requirements again, plusuvitself if it is not onPATH(infx/bench/uv.py:21-32). infx/tests/bench/test_agentic_runtime.py:70-97pins the rebuild-and-override behaviour.
The older bash path had the same shape: 408c015 inferencex-e2e/benchmarks/benchmark_lib.sh:3133 deletes the venv and :3140-3142 force the cache path; 3d55815 made AIPERF_UV_CACHE_DIR overridable (benchmarks/benchmark_lib.sh:1787-1790) but still deleted the venv (:1841). Main dropped that override.
Impact
For a caller such as Magpie, the client starts after the server is healthy, so the download runs while the server holds its GPUs and counts against the client timeout. On a slow or restricted network a run can fail for reasons unrelated to what is being benchmarked.
Proposed change (all in infx/bench/agentic/venv.py, plus tests)
- Honour the caller's cache: use
os.environ.get("UV_CACHE_DIR")when set, falling back to<root>/uv-cache. - Reuse a matching venv: key it on
PYTHON_VERSIONplus the bytes ofREQUIREMENTS, store the key in the venv after a successful install, and skip the rebuild when the key matches andagentxandhfare executable. Otherwise rebuild as today. - If the venv is meant to be shared across concurrent runs through a persistent
AIPERF_RUNTIME_DIR, take a file lock around the check-and-build. - Tests: a matching key makes no
uvcalls; changed requirements rebuild;UV_CACHE_DIRfrom the caller is used.
- Lenguaje dominante
- Python
- Estrellas
- 1.9k
- Forks
- 343
- Merge medio
- 2 d 40 min
- PR fusionados (30 d)
- 431
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de SemiAnalysisAI/InferenceX
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
SemiAnalysisAI/InferenceX#2125 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
SemiAnalysisAI/InferenceX#1587 ·
Los mantenedores suelen responder en 1 día
-
[experimental chore/agentx-v0.2-aiperf-testing branch] agentic build_replay_cmd: missing --tokenizer-trust-remote-code breaks Kimi-K2.5 (and other custom-code tokenizer models) / agentic build_replay_cmd 缺少 --tokenizer-trust-remote-code 导致 Kimi-K2.5 等自定义 tokenizer 模型失败Quizá libre de nuevo @sahil-bhatti-ai la tomó hace 169 días y no hay ningún pull request abierto. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
SemiAnalysisAI/InferenceX#1369 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 76/100
SemiAnalysisAI/InferenceX#1359 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 Menos de una hora Aptitud para principiantes 18/100
SemiAnalysisAI/InferenceX#4655 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de SemiAnalysisAI/InferenceX
Issues similares
-
getchaintips leaves out the active tip once a header is staged on it, where Core always lists itAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
btclib-org/btclib-node#2158 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
dbt-msft/dbt-sqlserver#897 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Gathin23/cleanyourtv#2770 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
vllm-project/vllm#64845 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
dekobon/big-code-analysis#1729 ·
Los mantenedores suelen responder en 1 día