-
Notifications
You must be signed in to change notification settings - Fork 49
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
Test case average of solutions in real dataset #15
Comments
We filtered when we collected the solutions to check if they pass the testcases however they may be horribly inefficient such as requiring several gigabytes of RAM to execute or take a really long time to execute (sometimes minutes). This varied based on the source of the ground truth solutions. We didn't filter the solutions further to only those that were optimal or near optimal. |
I see. I was running evaluation by generating code by simply copying first ground truth solution this way:
And when I run evaluation for these codes, I only got 60%. Does this seem right? |
You may need to select a different solution to test out. I can rerun the evaluation script to see how many optimal solutions. I might not be able to get to it for a while though as I'll need the compute in the background to re-evaluate all of the solutions and with sufficient RAM etc. So I can't really give an ETA on that. |
Okay, btw I see that the low 60% is due to some problems with '' character used unnecessarily in solutions at some places. |
Update: Doing sol = sol.replace('\n', '\n').replace('\"','"').replace('\r','').replace('\\','\').replace('\t','\t') has pushed it to >95% when I tested on first few training samples. |
Great thanks for the information. I thought we did that for our preprocessing but maybe something happened where it got removed. Feel free to make a PR with the changes if you have time :) |
Hi this isn't an error in the dataset, to load it in HuggingFace hub and respect some format constraints we had to save the solutions and input_output columns in json format which led to this behaviour. But in the README of the dataset we show how to load the solutions and input_output columns correctly: https://huggingface.co/datasets/codeparrot/apps#how-to-use-it |
@loubnabnl Thanks for the input! Leaving the issue closed. |
Hi, Have you checked the ground truth solutions in the original dataset to make sure that they pass the test cases?
The text was updated successfully, but these errors were encountered: