CSC/ECE 517 Fall 2020 - E2086. Let course staff as well as students do reviews: Difference between revisions
(→Issue) |
|||
Line 44: | Line 44: | ||
== Issue == | == Issue == | ||
=== | === Issues need to be fixed === | ||
[[File:E2086 assignment.png|1200px]] | |||
[[File:E2086 view submission.png|1200px]] | |||
[[File:E2086 submission.png|1200px]] | |||
== Test Plan == | == Test Plan == |
Revision as of 20:45, 22 October 2020
Project Index
- Deployment Link:
- Github Repository:
https://github.com/joeyzhou826/expertiza
- Github Pull Request:
- Video Link:
Project Background
- Expertiza Project - E2086 : Let course staff as well as students do reviews
Why This Project
Peer review is a great way for students to learn about how well they have developed their application. However, sometimes, the peer reviews may not be thorough and the team/person's work reviewed might not reflect the actual status of the development. By letting course staff perform reviews as well, the reviewer and the reviewee both benefit, improving the overall learning experience. Currently, there exists no method for the instructor/TA to perform reviews of assignments submitted by the students. Hence this is what this project aims at, allowing instructor/TA to perform reviews.
Primary Goal of this Project
This project aims to allow instructors to submit reviews of student work, using the same review form that students use to do reviews
Files Involved
list_submissions.html.erb
_calibration_report.html.erb
response.html.erb
Project Team Member
- Chenwei Zhou (czhou6)
- Tianrui Wang (twang33)
- Zhuolin Li(zli82)
- Hao Zhang (hzhang62)
Project Design Philosophy
Our project aims at enabling the instructor/TA to review the submissions using the same form that a student uses while peer-reviewing. For this, our implementation has been divided into 3 parts, and can be described as follows:
- The first part involves the primary functionality of letting the staff perform a review on students submission.
- The second part involves allowing the students to identify from their end, which review has been performed by an instructor/TA. This will help students improve their project as the opinions presented by a staff member would be valid.
- Third, we plan to enable the instructor/TA to do review instead of assign grade in the review round. After the review Round, the review review session will be closed and replaced with assigning grade interface
Issue
Issues need to be fixed
Test Plan
Refactor & Code Modification
Running Tests
rspec spec/models/response_spec.rb
Rspec Test Cases
Reference & Links
Previous Implementation
This project was done by Team E1985 on Fall 2019 with the following work:
Expertiza wiki: http://wiki.expertiza.ncsu.edu/index.php/CSC/ECE_517_Fall_2019_-_E1985._Let_course_staff_and_students_do_reviews
Github Pull Request: https://github.com/expertiza/expertiza/pull/1631
Github repository: https://github.com/Matrawick/expertiza
Youtube screencast: https://youtu.be/kLfZ22vlu30
Comments From Mentor
- Code for testing whether a reviewer is a staff member is put into an unrelated class (same method already exists elsewhere in system).
- There is a long code clone because the process of creating a mapping and a review is not quite the same as when the reviewer is a non-instructor; a lot of checks needed in that case are not needed here. But the current code should have been refactored to put the common code into another method.
- One test has been removed because it broke the build, & was evid. unrelated to their functionality.
- The commit was too messy; if this had been merged, it would have required refactoring.
for more information about this project, please visit Project Lists