-
Notifications
You must be signed in to change notification settings - Fork 61
Gradescope update #2053
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
base: dev
Are you sure you want to change the base?
Gradescope update #2053
Conversation
|
The logic in the gradescope directory are LLM translations of the prior perl scripts |
src/gradescope/autograders/README.md
Outdated
| @@ -0,0 +1,110 @@ | |||
| # Python Autograder System | |||
|
|
|||
| This directory contains the Python equivalent of the Perl-based Gradescope autograder system. | |||
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.
there is no Perl-based system in the repo anymore
INSTRUCTORS.md
Outdated
| 3. Update the `src/haz3lschool/Specs.re` module with `<module_name>.exercise`. | ||
|
|
||
| 4. Run `dune exec ./src/haz3lschool/gradescope.exe <path_to_student_json>` under project root to print the grade report. | ||
| 2. Run `make gradescope ZIP=<path to submissions zipfile> COURSE=<course_id> ASSIGNMENT=<assignment_id>` under project root to generate grade reports and upload them to Gradescope. |
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.
add instructions or pointer to the README for how to print the autograder report to the terminal rather than immediately sending it out to Gradescope
src/gradescope/autograders/README.md
Outdated
| 4. **Rich Ecosystem**: Easy access to scientific/data processing libraries | ||
| 5. **JSON Native**: Built-in JSON handling | ||
|
|
||
| ## Migration Notes |
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.
not necessary
src/gradescope/autograders/README.md
Outdated
| } | ||
| ``` | ||
|
|
||
| ## Advantages of Python Version |
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.
not necessary
src/gradescope/single_upload.py
Outdated
| @@ -0,0 +1,100 @@ | |||
| #!/usr/bin/env python3 | |||
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.
move to a gradescope subdirectory of a base grading directory
src/gradescope/autograders/AG.zip
Outdated
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.
this needs to be generated with a make target, rather than included as a binary blob
|
It currently can grade individual submissions. I think we should merge it so the grading logic remains in sync with dev. The missing features are batch grading, export to gradescope, and autograder generation, and these are hard to fix without access to submissions that are in-sync with current dev (last semesters I think are using different exercise formats), so I propose that we just wait until we have such submissions and then port over these capabilities from the previous commit. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #2053 +/- ##
==========================================
- Coverage 49.87% 49.80% -0.07%
==========================================
Files 218 218
Lines 23103 23103
==========================================
- Hits 11522 11506 -16
- Misses 11581 11597 +16 🚀 New features to boost your workflow:
|
Updated gradescope related logic in dev