Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Sep 6, 2025

Important

Add model information to eval runs table and update UI to display it, with refactoring and database schema changes.

  • Database:
    • Add columns name, contextWindow, inputPrice, outputPrice, cacheWritesPrice, cacheReadsPrice to runs table in 0002_bouncy_blazing_skull.sql.
    • Update schema.ts to reflect new columns in runs table.
  • UI:
    • Refactor Evals component in evals.tsx to use new EvalRun type and display additional model info.
    • Move plotting logic to new Plot component in plot.tsx.
  • Types:
    • Add EvalRun type in types.ts to include new fields.
  • Misc:
    • Add db:test:migrate and db:production:migrate scripts to package.json.
    • Minor comment fix in use-open-router-models.ts.

This description was created by Ellipsis for bcd31d6. You can customize this summary. It will automatically update as commits are pushed.

@cte cte requested review from jr and mrubens as code owners September 6, 2025 22:24
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. enhancement New feature or request labels Sep 6, 2025
Copy link
Contributor

@roomote roomote bot left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution! I've reviewed the changes and found some issues that need attention. The most critical is that this PR has merge conflicts that must be resolved before it can be merged.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Sep 6, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 6, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2025

🚀 Preview deployed!

Your changes have been deployed to Vercel:

Preview URL: https://roo-code-website-5ghyz724g-roo-code.vercel.app

This preview will be updated automatically when you push new commits to this PR.

@cte cte merged commit 247da38 into main Sep 7, 2025
12 checks passed
@cte cte deleted the cte/evals-model-info branch September 7, 2025 05:10
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Sep 7, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Sep 7, 2025

return (
<>
<div className="pt-4 pb-8 font-mono">Cost x Score</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider wrapping the user-facing header text ('Cost x Score') in a translation function (e.g., t()) to support internationalization.

Suggested change
<div className="pt-4 pb-8 font-mono">Cost x Score</div>
<div className="pt-4 pb-8 font-mono">{t("Cost x Score")}</div>

This comment was generated because it violated a code review rule: irule_C0ez7Rji6ANcGkkX.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.