Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Support Logic Reasoning Benchmark #1973
Support Logic Reasoning Benchmark #1973
Changes from 12 commits
c4d6686
d31e674
6e9339c
619d062
c255b75
be18cf2
5548f8d
4ad2147
d2f1d87
3f17da1
fe8b03f
ccae1d2
17a1ec2
ee94c24
c353e86
fd8b5e0
aeacbdd
951e420
7024342
7d80f4f
98c21b5
9917250
3b7e9d8
79a2240
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
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.
One tip: we have a
sandbox
parameter in main function, and you can execute the installation at here, then it may save some cost when you call gpt. At least they can reduce one action. But it is also fine to tell gpt in instructions.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.
thanks for the tip, any example for this? byw, currently i deprecated the sandbox in the main function.
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.
Here's an example (kudos to @ryanhoangt)
https://github.com/ryanhoangt/OpenDevin/blob/dce15b51237a7e9bbd399158a1c0c6536937f7d3/evaluation/mint/run_infer.py#L118-L130
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.
Thanks! already integrated! Sandbox is awesome, saving one action step for each instance!