Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Apr 3, 2025

Context

Screenshot 2025-04-11 at 10 19 04 PM

Implementation

Screenshots

before after

How to Test

Get in Touch


Important

Adds concurrency control for evals in the web app, including database schema updates, CLI modifications, and a new UI slider component.

  • Concurrency Control:
    • Adds concurrency field to runs table in database (0002_white_flatman.sql, schema.ts).
    • Updates createRunSchema in schemas.ts to include concurrency with min, max, and default values.
    • Modifies run() in index.ts to use run.concurrency for task execution.
  • Web App UI:
    • Adds Slider component in slider.tsx for setting concurrency in new-run.tsx.
    • Updates new-run.tsx to include concurrency slider in the form.
  • Package Updates:
    • Adds @radix-ui/react-slider to package.json dependencies.
    • Updates various devDependencies in package.json.

This description was created by Ellipsis for b6c82dc. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Apr 3, 2025

⚠️ No Changeset found

Latest commit: b6c82dc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Apr 3, 2025
@hannesrudolph hannesrudolph moved this from New to PR [Pre Approval Review] in Roo Code Roadmap Apr 6, 2025
@mrubens
Copy link
Collaborator

mrubens commented Apr 7, 2025

This does seem nice to have!

@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Apr 11, 2025
cte and others added 2 commits April 11, 2025 15:07
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Apr 12, 2025
@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Apr 12, 2025

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at [email protected]


Generated with ❤️ by ellipsis.dev

@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Apr 12, 2025

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at [email protected]


Generated with ❤️ by ellipsis.dev

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XXL This PR changes 1000+ lines, ignoring generated files. labels Apr 12, 2025
@cte cte merged commit 6ab9aa9 into main Apr 12, 2025
14 checks passed
@cte cte deleted the cte/evals-concurrency branch April 12, 2025 05:21
@github-project-automation github-project-automation bot moved this from PR [Pre Approval Review] to Done in Roo Code Roadmap Apr 12, 2025
JorkeyLiu pushed a commit to JorkeyLiu/Roo-Code that referenced this pull request Apr 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants