-
Notifications
You must be signed in to change notification settings - Fork 8
Stanford cs107 fall 2023 final #53
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
|
Could you also share the original exam? |
|
@tareknaser Would you like me to upload the pdf to the same directory of |
Yes please |
|
@tareknaser it's done |
tareknaser
left a comment
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.
Thank you! I reviewed this and everything makes sense. it’s clear that it’s well thought out. I have minor comments and a couple of questions but I think this is very close to be merged
benchmarks/courseexam_bench/data/cs107_computer_organization_&_systems_fall_2023_final/exam.md
Outdated
Show resolved
Hide resolved
benchmarks/courseexam_bench/data/cs107_computer_organization_&_systems_fall_2023_final/exam.md
Outdated
Show resolved
Hide resolved
benchmarks/courseexam_bench/data/cs107_computer_organization_&_systems_fall_2023_final/exam.md
Show resolved
Hide resolved
benchmarks/courseexam_bench/data/cs107_computer_organization_&_systems_fall_2023_final/exam.md
Show resolved
Hide resolved
benchmarks/courseexam_bench/data/cs107_computer_organization_&_systems_fall_2023_final/exam.md
Show resolved
Hide resolved
benchmarks/courseexam_bench/data/cs107_computer_organization_&_systems_fall_2023_final/exam.md
Show resolved
Hide resolved
benchmarks/courseexam_bench/data/cs107_computer_organization_&_systems_fall_2023_final/exam.md
Show resolved
Hide resolved
|
@tareknaser a new commit dealing with the comments above is pushed |
tareknaser
left a comment
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 nice work
* modified the regular expression match * added cs107_fall_2023_final * updated one wrong tag * Added pdf of the paper * solved all the comments
Description
Added final exam of cs107 in fall 2023, modified the regular expression match when extracting json metadata of exams, as current one can't deal with situations where
answerfield of a question's metadata contains code block with '`'Changes
prepare_dataset.pyTesting
Ran
benchmarks/courseexam_bench/tests/test_data_schema.pyChecklist