CSC/ECE 517 Fall 2019 - E1985. Let course staff and students do reviews: Difference between revisions
| Line 16: | Line 16: | ||
=== Allow TA/Instructor to add himself as a participant === | === Allow TA/Instructor to add himself as a participant === | ||
Currently instructors and TA's can add themselves as participants. The teaching staff reviewing a previous project thought this might be a bad idea. Here's what they said about it in our project description. "It adds instructor to the assgt. as a participant. This is probably not a good idea, because the instructor can review everyone, independent of what reviewer-assignment mechanism was in use." | Currently instructors and TA's can add themselves as participants. The teaching staff reviewing a previous project thought this might be a bad idea. Here's what they said about it in our project description. "It adds instructor to the assgt. as a participant. This is probably not a good idea, because the instructor can review everyone, independent of what reviewer-assignment mechanism was in use." | ||
Revision as of 01:26, 11 November 2019
Problem Statement
Peer review is a great way for students to learn about how well they have developed their application. However there are some problems with this:
- Sometimes, the peer reviews may not be thorough and the team/person's work reviewed might not reflect the actual status of the development.
- The reviewer might not know how well they are reviewing the peers work. They might not entirely know what tone to use or what suggestion to make.
By letting course staff perform reviews as well, the reviewer and the reviewee both benefit, improving the overall learning experience.
Current scenario
Issue : Method to check if user is instructor / TA for selected course
Currently, there is no method which checks if the user is an instructor / TA for the selected course (namely a course staff member) Need : This method is required for redirection of pages which is dependent on the user role. Also, it is required to identify if the review is performed by the course staff. Proposed implementation : A new helper file named course_staff_helper.rb will be created in expertiza/helpers/ which will consist a method which checks if the user is a course staff member.
Allow TA/Instructor to add himself as a participant
Currently instructors and TA's can add themselves as participants. The teaching staff reviewing a previous project thought this might be a bad idea. Here's what they said about it in our project description. "It adds instructor to the assgt. as a participant. This is probably not a good idea, because the instructor can review everyone, independent of what reviewer-assignment mechanism was in use."