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

grading: Add lab grading script #6

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

Conversation

teodutu
Copy link

@teodutu teodutu commented Oct 9, 2023

This script copies students' grades from a given attendance sheet to the course's register. It requires unique IDs (such as LDAP accounts) to avoid overlaps given by other forms of identification.

@teodutu teodutu force-pushed the add-lab-grading branch 2 times, most recently from b0f9d9e to d305589 Compare October 9, 2023 20:44
Copy link

@Alex-deVis Alex-deVis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left you two suggestions, I will test this tomorrow.

For the issue with sheets' name restrictive pattern (e.g. Lab X) you could retrive all sheet names and filter them using a pattern.

grading/grade-labs/course_registers.json Show resolved Hide resolved
Returns one ValueRange object that writes the given value at col[idx].
"""
pos = col.find(":")
# TODO: [Bug] This assumes actual grades start from a row < 10.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like something a regex would fix.

@teodutu
Copy link
Author

teodutu commented Oct 9, 2023

For the issue with sheets' name restrictive pattern (e.g. Lab X) you could retrive all sheet names and filter them using a pattern.

Sure, is it acceptable to request that the string Lab <num> be present in the sheet's name?

@Alex-deVis
Copy link

Alex-deVis commented Oct 9, 2023

Sure, is it acceptable to request that the string Lab  be present in the sheet's name?

Yes, just make sure you mention this in the README.md or argparse help or throw an ugly exception if no sheet is found.

@teodutu teodutu force-pushed the add-lab-grading branch 3 times, most recently from d7b7fe3 to 09355c4 Compare October 21, 2023 10:17
@teodutu teodutu force-pushed the add-lab-grading branch 2 times, most recently from 5cadd76 to ba855e9 Compare March 5, 2024 20:46
This script copies students' grades from a given attendance sheet to the
course's register. It requires unique IDs (such as LDAP accounts) to avoid
overlaps given by other forms of identification.

Signed-off-by: Teodor Dutu <[email protected]>
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.

2 participants