CSC/ECE 517 Fall 2021 - E2164. Heatgrid fixes and improvements: Difference between revisions
Line 22: | Line 22: | ||
== Test Plan == | == Test Plan == | ||
== Files to be | == Files to be Targeted == | ||
<li> app/controllers/grades_controller.rb:- Focussing on self-review and grades section. </li> | |||
<li> app/views/_review_strategy.html.erb:- Here, the checkbox of "Allow Self Review?" is written. </li> | |||
<li> app/helpers/grades_helper.rb:- Grades helper file. </li> | |||
<li> spec/controllers/review_mapping_controller.rb:- Adding test cases for checking whether the reviewer can self-review if the "Allow self reviews" checkkbox is disabled and the vice-versa too. </li> | |||
=Issue 2= | =Issue 2= |
Revision as of 00:09, 4 November 2021
Project Background
Heatgrid is the page where instructor can visit and view the work done by participant using Reviews, Author Feedbacks and Meta Reviews. This one web page displays all the evaluations done by peers of the participant for his project. The page is displayed in the format of table with colors. The columns in the table can be sorted with the requirement of the project. Heatgrid shows scores assigned by reviewers on individual rubric items for participant.
What is wrong with it
It is called from Instructor's "Assign Grades" function and student's view scores functions. But , there are issues that students are reviews themselves. Question is about Are reviews getting assigned correctly? Assignement of metric is also wrong at some places
Issue 1
Problem Definition
Issue #2019
Questions whether the reviewers are correctly identified. In one case, a student was shown as reviewing his/her team, which is impossible. Also write tests so that if this bug occurs again, we will be informed.
Current Implementation
Overview
UI Screenshots
Control Flow
Observations
Test Plan
Files to be Targeted
Issue 2
Problem Definition
Issue#1869
Notes that there is a “metric-1” column in the review report, but it doesn’t say what the metric is. Please figure out what the metric is, and change the column header accordingly. However, the intent was that eventually an instructor would be allowed to select a metric to be shown on the heatgrid. So, somewhere in the UI for creating/editing an assignment, there should be a way for the instructor to select a metric from a dropdown list. You don’t have to add any more metrics to this list other than the one that is currently displayed, just create a mechanism for adding metrics in the future.
Current Implementation
Overview
UI Screenshots
Control Flow
Refactoring
Files to be Edited
Test Plan
Important Links
Team
- Shlok Sayani (sdsayani)
- Hardik Udeshi (hvudeshi)
- Isha Gupta (igupta)
- Manish Shinde (msshinde)