secretary
/
waitonbot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed point distribution

This commit is contained in:
Jacob Henry 2018-12-02 12:36:02 -05:00
parent c5231f1ef0
commit cdc3cee931
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
house_management.py
View File

@ -261,9 +261,10 @@ def apply_house_points(points: List[PointStatus], assigns: List[Optional[JobAssi
Destroys existing values in the column.
Should be called each time we re-export, for validations sake.
"""
# First, eliminate all house points
# First, eliminate all house points and signoff points
for p in points:
p.job_points = 0
p.signoff_points = 0
# Then, apply each assign
for a in assigns:
@ -273,19 +274,19 @@ def apply_house_points(points: List[PointStatus], assigns: List[Optional[JobAssi
# What modifier should this have?
if a.signer is None:
score = MISS_VAL
job_score = MISS_VAL
else:
if a.late:
score = LATE_VAL
job_score = LATE_VAL
else:
score = JOB_VAL
job_score = JOB_VAL
# Find the corr bro in points
for p in points:
# If we find, add the score and stop looking
# If we find assignee, add the score, but don't stop looking since we also need to find signer
if p.brother == a.assignee:
p.job_points += score
break
p.job_points += job_score
# If we find the signer, add a signoff reward
if p.brother == a.signer:
p.signoff_points += SIGNOFF_VAL