Skip to content

fix(ci): make refresh dispatch explicit about evals and experiments - #362

Merged
mattrossman merged 8 commits into
mainfrom
mattrossman/ai-1005-intersect-explicit-experiments
Oct 6, 2026
Merged

mattrossman merged 8 commits into
mainfrom
mattrossman/ai-1005-intersect-explicit-experiments

Conversation

@mattrossman

@mattrossman mattrossman commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Dispatching a refresh for specific experiments has failed several times since July (Jul 16, Jul 24, Jul 28, Aug 7, Aug 14, Aug 26, Sep 9, Sep 30), and keeps coming up on Slack. This causes false alarms on runs and wasted sandboxes.

Manual refresh dispatches now run only what you ask for:

  • Explicit experiments run only in suites they belong to. Each listed experiment used to be planned in every suite the eval allows. A mixed list now plans just the 2 valid pairs.
  • No benchmark default. suite and experiment_suite start blank, and the run fails fast unless you name evals (suite or eval) and experiments (experiment_suite or experiments). The default made sense when refreshes were mainly handled by AI team and mostly for the public benchmark, but now the benchmark is rarely refreshed and multiple teams work in the repo. A dispatch with only eval set and suite blank still plans its pairs.
  • One refresh branch per manual run. create-pull-request resets an existing branch, so a shared name let a later run wipe an earlier run's results. Scheduled runs still share one branch. To keep adding to one refresh PR, dispatch on its branch with commit_to_branch=true.
  • merge now defaults to true. This is the most common way we dispatch refreshes manually, and merging is necessary for the recommended workflow of accumulating multiple refreshes on a single branch with commit_to_branch=true

Closes AI-1005

@vercel

vercel Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
evals Ignored Ignored Preview Oct 6, 2026 1:28pm UTC

Request Review

@mattrossman mattrossman changed the title fix(ci): support mixed explicit refresh experiments fix(ci): make refresh dispatch explicit about evals and experiments Oct 5, 2026
@mattrossman
mattrossman marked this pull request as ready for review October 5, 2026 18:13
@mattrossman
mattrossman requested a review from a team as a code owner October 5, 2026 18:13
Comment thread .github/workflows/eval-refresh.yml Outdated
Comment thread .github/workflows/eval-refresh.yml

@Rodriguespn Rodriguespn 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.

Approving with two comments. Please read them before merging. Thank you for working on this!

@mattrossman

Copy link
Copy Markdown
Collaborator Author

Updated!

@mattrossman
mattrossman merged commit 6e966c2 into main Oct 6, 2026
7 checks passed
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.

2 participants