-
Notifications
You must be signed in to change notification settings - Fork 6.9k
feat(aci): adding file viewing capability to different extension types #8742
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.
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
Conversation
Baseline performance with claude 4 sonnet:
|
With changes:
I'm gonna run a larger set (200) to make sure |
On main: 133/200 Let's go with this @ryanhoangt |
ec52897
to
9e5dd86
Compare
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like
Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
With changes (with updated prompt): 134/200
|
@ryanhoangt I think we can cut a new release in openhands-aci and get it merged |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
End-user friendly description of the problem this fixes or functionality this introduces.
Summarize what the PR does, explaining any non-trivial design decisions.
Part of changes in #8598. This PR contains changes to the str_replace_editor
Link of any specific issues this addresses:
To run this PR locally, use the following command: