Skip to content
Snippets Groups Projects
Commit 3878dc42 authored by Christopher Bohn's avatar Christopher Bohn :thinking:
Browse files

Added a couple of TODOs

parent c9f8b987
No related branches found
No related tags found
No related merge requests found
......@@ -19,6 +19,7 @@ def strip_html(text: Optional[str]) -> Optional[str]:
else:
return None
def semester_stamp() -> str:
today = datetime.date.today()
year: int = today.year
......
......@@ -40,7 +40,7 @@ def get_project_prefix(canvas_groups):
def display_git_contributions(project: GitlabProject):
# TODO: recognize that this only works for projects in namespace; will need to ask whether to retrieve project.
commits: List[GitlabCommit] = project.get_commits() # TODO: narrow the selection
contributions: Dict[str, int] = {}
contributions: Dict[str, int] = {} # TODO: also broaden to multiple branches?
contributors: Set[Tuple[str, str]] = set()
# noinspection PyShadowingNames
for commit in commits:
......@@ -89,9 +89,9 @@ if __name__ == '__main__':
zero_padding: int = ceil(log10(len(projects)))
for student_group in student_groups: # TODO: Skip past graded groups
input(f'\n\nPress Enter to grade {student_group}')
print()
project_name = f'{project_prefix}{student_group.get_name().split()[1]}'.zfill(zero_padding)
display_git_contributions(list(filter(lambda p: p.get_name() == project_name, projects))[0])
print()
display_peer_reviews(peer_review_assignment, student_group.get_students())
# TODO: Ask if you want to grade (keep track of groups that you don't grade)
if True:
......
......@@ -25,6 +25,7 @@ def get_students() -> Set[CompositeUser]:
# TODO: assign_partners for arbitrarily-sized teams
# TODO: break this up into bite-sized chunks
# TODO: manage multiple sections
def create_pairs(students: Set[CompositeUser], groupset_name: str = 'Unknown Assignment') -> \
List[Tuple[int, CompositeUser, CompositeUser, Optional[CompositeUser]]]:
students_with_blacklist: Set[CompositeUser] = sorted(list(filter(lambda s: s.has_blacklist(), students)),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment