Skip to content

Local Evals#828

Open
gabeweisz wants to merge 24 commits into
mainfrom
feat/gw_pi_evals
Open

Local Evals#828
gabeweisz wants to merge 24 commits into
mainfrom
feat/gw_pi_evals

Conversation

@gabeweisz

Copy link
Copy Markdown
Collaborator
  • Modify the repeatability test script to be able to run against a local Pi server
  • Create a script to run a local server (usually vllm but sglang should work), install and configure Pi, and run local evals using it

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for running the repeatability analysis/eval harness against a local “pi” agent backend, and introduces a Docker-based launcher to spin up a local OpenAI-compatible inference server, configure pi from /v1/models, and run the evals end-to-end.

Changes:

  • Extend run_repeatability_parallel.sh to support --pi / USE_PI=1, unify agent invocation, and improve TEST_IDS filtering + reporting.
  • Add run_pi_analysis_in_docker.sh to launch an inference server in Docker, install/configure pi + TraceLens, and run the repeatability harness using pi.
  • Update generate_ref.sh TEST_IDS filtering to support underscore-prefix matching and warn when no IDs match.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
agent_evals/Analysis/eval_scripts/run_repeatability_parallel.sh Adds pi backend support, refactors agent invocation, improves test filtering/logging, and normalizes trace/reference paths.
agent_evals/Analysis/eval_scripts/run_pi_analysis_in_docker.sh New Docker launcher script to start a local inference server, configure pi, install TraceLens, and run evals.
agent_evals/Analysis/eval_scripts/generate_ref.sh Improves TEST_IDS filtering semantics and adds a “no matches” warning.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread agent_evals/Analysis/eval_scripts/run_repeatability_parallel.sh Outdated
Comment thread agent_evals/Analysis/eval_scripts/run_repeatability_parallel.sh Outdated
Comment thread agent_evals/Analysis/eval_scripts/run_repeatability_parallel.sh Outdated
Comment thread agent_evals/Analysis/eval_scripts/run_pi_analysis_in_docker.sh Outdated
gabeweisz and others added 6 commits July 20, 2026 10:50
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@Ahmedhasssan-aig Ahmedhasssan-aig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

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.

3 participants