Skip to content

assistant_eval: Add ACE framework #27181

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
tmickleydoyle merged 29 commits into from
Apr 3, 2025
Merged

assistant_eval: Add ACE framework #27181

tmickleydoyle merged 29 commits into from
Apr 3, 2025

Conversation

@tmickleydoyle
Copy link
Contributor

@tmickleydoyle tmickleydoyle commented Mar 20, 2025 •
edited
Loading

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Mar 20, 2025
@tmickleydoyle tmickleydoyle marked this pull request as draft March 20, 2025 12:12
@maxdeviant
Copy link
Member

maxdeviant commented Mar 20, 2025

Release Notes:

  • Update the assistant2 panel evaluation workflow to work with more coding examples

Just a note that we only need to include release notes for things that we want to show up in the release notes.

For internal changes or for things that aren't released yet, we can just put:

Release Notes:

- N/A

maxdeviant
maxdeviant reviewed Mar 20, 2025
View reviewed changes
@tmickleydoyle tmickleydoyle marked this pull request as ready for review March 28, 2025 17:35
@maxdeviant maxdeviant changed the title Thomas/ace framework assistant_eval: Add ACE framework Mar 28, 2025
@tmickleydoyle tmickleydoyle merged commit cd85b43 into main Apr 3, 2025
16 checks passed
@tmickleydoyle tmickleydoyle deleted the thomas/ace-framework branch April 3, 2025 04:02
JosephTLyons pushed a commit that referenced this pull request Apr 4, 2025
@tmickleydoyle @mgsloan @JosephTLyons
assistant_eval: Add ACE framework (#27181)
b037c91 
Release Notes:

- N/A

---------

Co-authored-by: Michael Sloan <michael@zed.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maxdeviant maxdeviant maxdeviant left review comments

+1 more reviewer

@mgsloan mgsloan mgsloan left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@tmickleydoyle tmickleydoyle

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tmickleydoyle @maxdeviant @mgsloan