CSC/ECE 517 Spring 2018- Project E1824: Let course staff as well as students do reviews: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
Line 60: Line 60:
[[File: Grades_view_team_instr.png | center]]
[[File: Grades_view_team_instr.png | center]]


A student should be able to make out if an instructor has reviewed his/his team's work. In case the instructor performs a review on the team's work, it will highlight the intructor's review.
A student should be able to make out if an instructor has reviewed his/his team's work. In case the instructor performs a review on the team's work, it will highlight the instructor's review.
 
 
'''Step 3: If instructor has reviewed the same work as you have reviewed, provide a way to show his review as well in ''Others work'''''
 
 
Files to be edited:
* '''View: ''' ''app/views/student_review/list.html.erb''
* '''Controller: ''' ''app/controllers/student_review_controller.rb'' - Method: view_team
 
[[File: Others_work_proposed.jpeg | center]]
 
If the instructor has reviewed the work of a team you have reviewed, there will be a link to view how the instructor has reviewed the same work. In case the instructor has not reviewed the work (like in case of review 4), there won't be any ''Instructor's Review'' link


== '''Test Plan''' ==
== '''Test Plan''' ==

Revision as of 05:04, 29 April 2018

Introduction

The way Expertiza is set up right now is that only peers can review your work. However, there are cases when the instructor would want to submit reviews as well. This project aims to implement this feature by allowing instructors to review the project on the same metrics as other students who review the project.


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 peer's work. They might not entirely know as to what tone to use or what suggestions to put forward.

By letting course staff perform reviews as well, the reviewer and the reviewee are both benefited, which can improve the overall learning experience.


Current scenario

This is how some of the pages we are concerned with, currently look.


Submissions page for an assignment in Instructor View


A typical Scores table in a Student View

Proposed Solutions

We propose to perform the following changes to let staff perform reviews as well:

Step 1: Add a Perform review link in the assignment submissions page view for the instructors.


Files to be edited:

  • View: app/views/assignments/list_submissions.html.erb
  • Controller: app/controllers/response_controller.rb - Add authorization for instructors to perform review

In the second case, we can see a 'Perform Review' link. This is the initial state when instructor review has not been added to the submission. Once the instructor adds a review, we can see the 'View Review' and 'Edit Review' links, as seen in the first case. If the review deadline of one round is passed, the 'Edit Review' link becomes 'Update Review' link. The course staff will be allowed to perform a review even if the deadline for reviewing has passed.

In the second case, an intermediate submission deadline has passed and hence there is a link only for performing a review.

In the third case, if no submission deadline has passed, there won't be a link for Performing a review nor assigning a grade. [No, it should be dependent on whether reviewing is allowed during the current assignment phase.]


Step 2: Add instructor review in Your Scores table in case he has reviewed your work. Provide a highlight/way to make it look distinct.


Files to be edited:

  • View: app/views/grades/view_team.html.erb
  • Controller: app/controllers/grades_controller.rb - Method: view_team

A student should be able to make out if an instructor has reviewed his/his team's work. In case the instructor performs a review on the team's work, it will highlight the instructor's review.

Test Plan

Many of our changes would be reflected on views (user interface). Following is the list of tests that should be conducted with rspec/capybara:

  1. it "should see the Perform Review link (beside Assign Grade link) for each submission after a list of submissions are pulled up by an instructor if the final due date for the assignment has passed"
  2. it "should see the Perform Review link only for each submission after a list of submissions are pulled up by an instructor if the final due date for the assignment has not passed and review period has started and review deadline has not passed"
  3. it "should see neither Perform Review link nor Assign Grade link after a list of submissions are pulled up by an instructor if the final due date for the assignment has not passed and review period has not started"
  4. it "should pull up a review page for the submission of the team after the Perform Review link is clicked on by an instructor"
  5. it "should show highlighted reviews done by instructors when a student views feedback on his/her submission"
  6. it "should pull up a calibration report when a student wants to see the difference between his/her reviews on other's submission and the instructor's"


External Links


Team Members

  • Ashis Sahoo
  • Satvik Shetty
  • Xiaohui Ellis
  • Kushal Nawalakha


References

Expertiza

Expertiza Github

Expertiza Documentation

Retrieved from "https://wiki.expertiza.ncsu.edu/index.php?title=CSC/ECE_517_Spring_2018-_Project_E1824:_Let_course_staff_as_well_as_students_do_reviews&oldid=117256"