Skip to content
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

Use student_identifier only to get files #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gklarenberg
Copy link

I used this package this past summer semester and made some changes/updates. When you download assignments from Canvas, it gives every file a unique number (aside from the student identifier) and also keeps the name that the students gave the file. So all files have different names. Same with when people submit assignments late via email.
So I added the option to search for assignment files by student_identifier only. You do still need to give the full file name of the first student (I'd like to change this too though).
I implemented the change by defining an assignment_folder = argument, which is the name of the folder where the assignment files are stored. This assumes that this folder only contains assignments. I can still work on error proofing this.
I also added an argument for the number of assignment parts, to make sure this feature doesn't break.
The main code changes are in core_assist_grading, with an if-else statement.
I am really new to making packages so I haven't updated any of the comments at the top of the scripts, but can do that if you prefer. I haven't tested this new code with the team grading option, or pushing things to GitHub...

… only; in case assignment names are not identical. This is done by defining an assignment_folder argument, where the assignment files should be stored. Also added an argument for number of assignment parts, to make sure this feature doesn't break. Main code changes are in core_assist_grading, with an if-else statement.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant