-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
small bug in sample logic eval #164
Comments
@DustinWehr I am willing to take this issue. May I work on it? |
@vedant-z There is nothing to work on. If one of the openai people says to do a PR, I'll do it. But they might just want to push a quick fix or add it to an internal documentation todo list. |
In general, feel free to open a PR for any issues you see in the repo. Thanks! |
andrew-openai
pushed a commit
that referenced
this issue
Mar 30, 2023
Just fixes the logic puzzle answer, which was a bit wrong. Close #164.
Linmj-Judy
pushed a commit
to TablewareBox/evals
that referenced
this issue
Feb 27, 2024
Just fixes the logic puzzle answer, which was a bit wrong. Close openai#164.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Set of pull requests seems to be growing pretty fast, so not confident I should add another for such a small thing.
Question is, "The day before yesterday, Chris was 7 years old...."
Minimal fix is to change the last "this" in the ideal answer to "next". Should be updated in evals/registry/data/README.md also.
The text was updated successfully, but these errors were encountered: