Skip to content

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
Collaborator

@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.

Copy link
Collaborator

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
Collaborator

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]>
@teodutu teodutu force-pushed the add-lab-grading branch from cb0e634 to 270b4ba Compare May 23, 2025 14:20
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