Skip to content

chore(data): seed corpus bundles (run 3090) - #1362

Open
github-actions[bot] wants to merge 1 commit into
developfrom
chore/seed-corpus-20260730-3090
Open

chore(data): seed corpus bundles (run 3090)#1362
github-actions[bot] wants to merge 1 commit into
developfrom
chore/seed-corpus-20260730-3090

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Seed corpus update

Automated PR from the seed-corpus workflow (run 3090).

Bundles included

New or updated result bundles are committed under results-data/bundles/.
Review the diff to confirm expected benchmark × platform × scale coverage.

Trust label

All bundles in this PR carry the maintainer-run / public-curated trust label.

Merging

Merge into develop triggers sync-results-data-to-published, which
opens a draft PR mirroring these bundles onto the published-results
corpus branch.

Copy link
Copy Markdown
Owner

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 19cc42bb69

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

},
"version": "1.40.0"
},
"queries": [],

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Remove the empty Polars SSB bundles

When these bundles reach the published-results mirror, the Validate bundles step in .github/workflows/validate-submission.yml rejects them because scripts/validate_submission.py requires summary.validation to be passed, while this artifact contains no query timings and reports not_run; the SF 0.1 Polars sibling has the same defect. Besides blocking publication, treating either artifact as Polars coverage would publish a run that executed none of SSB's queries, so omit them until SSB DataFrame queries are available.

Useful? React with 👍 / 👎.

@@ -0,0 +1,722 @@
{
"benchmark": {
"id": "ssb",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Regenerate the corpus inventory with the new bundles

Adding this bundle and the other five without updating results-data/corpus-inventory.json makes the Check corpus inventory step in .github/workflows/validate-submission.yml fail; running python scripts/generate_corpus_inventory.py --check reports all six new paths as missing. Even after removing the invalid Polars artifacts, the valid DataFusion and DuckDB additions still leave the mirror PR blocked, so regenerate and commit the inventory alongside the bundles.

Useful? React with 👍 / 👎.

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