sak007 / expertiza Goto Github PK

Missing/skipped assignments should be signified by a dash, not a hyphen (which is too hard to see) in course_student_grade_summary.

Missing/skipped assignments should be signified by a dash, not a hyphen (which is too hard to see).

all_students_all_reviews looks at metarviews and teammate reviews. It should be possible to view just one or the other. So the method should take a parameter specifying which kind(s) of review scores are to be displayed, and columns relating to reviews that are not being displayed should be suppressed, so they don’t appear in the listing. [Could be expanded to list peer-review averages as well.]

There is an issue with course ID 130 when it loads:
http://152.7.99.75:8080/assessment360/index?course_id=130

Please check.

We would like to have the following columns potentially shown, depending on whether the corresponding box is checked
i) Instructor-assigned scores
ii) Peer grades
iii) Teammate review scores
iv) Topics
v) Meta-review scores (only show if the set of deadlines has a meta-review deadline). find_due_dates(metareview) see the deadline_types table for the parameter name
vi) Number of teammates [cumulative]

The bullet gem should be used to minimize the number of db accesses.

course_student_grade_summary should show average peer-review scores, as well as instructor-assigned scores.

"If a student has not taken an assignment or if they have not received any grade for the same, assign it as 0 instead of leaving it blank. This helps in easier calculation of overall grade. Averages should only include assignments that were attempted."

Sure it’s easier, but it’s not accurate! Averages should only include assignments that were attempted.

@teamed_count is a cryptic name for a variable; rename. If it is not used, then simply remove it.