CSC/ECE 517 Fall 2022 - E2282. Fix teammate-review view: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
No edit summary
Line 178: Line 178:


[[File:yes_description_on_heatmap.png|center|border|1000px|Teammate Review Heatmap has descriptive title]]
[[File:yes_description_on_heatmap.png|center|border|1000px|Teammate Review Heatmap has descriptive title]]
=== Manual Testing for Anonymized User View: ===
The following tests were conducted as follows:
<br>
- Sign in as test student7185 above
<br>
- Navigate to assignments
<br>
- Click on the Final Project assignment
<br>
- Click on "Your scores" button
<br>
==== 1) View your teammates names in the Team members section of review scores  ====
[[File:Example.jpg]]
==== 2) Switch to Anonymized View and view your teammates names in the Team members section of review scores ====
[[File:Example.jpg]]

Revision as of 21:50, 30 November 2022

Problem Description

In Expertiza, both students and instructors are able to view a student's teammate reviews. On the reviews page, only a single heatmap is shown which does not provide a full representation of the student's performance. In addition, it is unclear if the heatmap is displaying information about the reviews the student has written or received. The heatmaps on the instructor and student views are not labeled and do not provide clarity on the information they are displaying.

Proposed Changes

Calculate Composite Review Score

Problem

It would be useful for the instructor to see a composite score derived from all of the reviews similar to the average score shown on the students review page. Currently, the code only displays the average score for each row of the review table on the instructors view. The image below shows where we would like to display the composite score.

Instructor view of a student's reviews does not show a composite score
Instructor view of a student's reviews does not show a composite score

Solution

The previous semester's team calculated the composite value within the view page which does not follow the design principles of this course. We plan to move this mathematical operation to the appropriate model where it where will be accessible to any view that requires a composite score for its heatmap.

The problem will have to be solved through changes in the two files listed below:

rscore.rb

This file contains the logic for doing mathematical functions for review tables. In the init function, the average score for each table is calculated. A new method can push the average score for each review into an array that can later be used to calculate the average across all reviews. Another method can reference this array to calculate the composite score across each review.

_view_heatgrid.html.erb

This file will call the composite score method described above to populate a string tag that displays the average score for all reviews on a view.


Improved Handling of Relevant Reviews

Problem

The heatmap displayed on the Student view does not give the full representation of the group performance. The view should show both the reviews received from a students teammates as well as external reviews. The image below shows that the students can not see their teammates reviews even when the instructor checkbox for viewing teammate reviews is selected.

The "Your Scores" page for students does not show teammate reviews
The "Your Scores" page for students does not show teammate reviews

Solution

We will need to improve how the review table(s) intended for display are generated.

The problem will have to be solved through changes in file listed below:

grades_controller.rb

This file contains the logic for orchestrating the build of the heatmap tables. The logic is this controller will need to be updated to populate the table array with reviews from each team participant.

view_team.html.erb

This file controls what tables are displayed when an instructor views a team. This view will needed to be updated to include the display the missing tables.


Improved Display of Review Heatmap

Problem

This problem contains many parts. Firstly, the student scores view does not even show a heatmap of reviews at all as seen below.

The my scores view page for the student doesn't show any teammate review heatmaps
The my scores view page for the student doesn't show any teammate review heatmaps

This will have to be included with the condition that the instructor has selected that the student can view the reviews in the assignment options. Additionally, in the display of the heatmap itself there is no indication if the reviews displayed are reviews about the student signed in or made by the student signed in as seen below.

The heatmap printout doesn't specify if showing reviews by or for a certain student
The heatmap printout doesn't specify if showing reviews by or for a certain student

A more descriptive label will have to be added to show this. Lastly, a composite score will have to be displayed under each heatmap to average the scores that the student received.

Solution

The problem will have to be solved through changes in two erb files listed below:

_view_heatgrid.html.erb

This file contains the logic to display the actual heatgrid to the user. It will have to be modified in two ways. Firstly, we will have to add a conditional header that will either display that the heatgrid shows reviews for the student or by the student depending on the type of questionnaire that it is displaying. Secondly, we will have to add a place under the heatgrid to display the composite score of that heatgrid. In the previous attempt at this project, the group added the calculation logic for the composite score to this file which is incorrect because it is an erb file. Instead, this file will contain a call to a funtion in the model that will calculate the value and return it for display.

view_team.html.erb

This file contains the logic to display the page where the student views the questionnaires. We will have to add logic to display the reviews of that student if the assignment is set to have show teammate reviews be true.


Preservation of Anonymized View Naming Convention

Problem

Student names are visible in the teammate reviews tab, even when the user is in the anonymized view.

Solution

We plan to remove the team member name from being displayed when using anonymized view by adding some changes to the following files.

user.rb

In user file the anonymized user function is defined and when called it checks and returns True if a particular user is in anonymized view or else returns False. We will be improving the conditional logic to display the user name only when not present in anonymized view.

_view_heatgrid.html.erb

The file contains the logic to display the heatgrid to the user. When we make the changes to user file we will need to make some changes in this file to not display the user name in anonymized view.

Bugs

In addition to our required changes, we plan to explore the following bugs that we have found on the reviews page for the instructor:

Up until student 9020 all scores and averages are correct. It is theorized that the average score for student 9020 is being pulled from student 8977's scores on table 1. Student 9020 is showing review scores of student 9022 so the cells have become shifted by 1. The student 9020 review has essentially disappeared from the view. The average of each row is being populated in student 9042 due to this shift.

The top image shows the student 9022 review which you can see is populated in student 9020 column from the bottom image.

Student 9022 review
Student 9022 review
Instructors reviews page should bugs on the table
Instructors reviews page should bugs on the table

Testing

Flow to access affected views

_view_heatgrid.html.erb

This file can be accessed through the GUI differently when logged in as an instructor or a student. When logged in as a student:

  1. Click "Assignments" at the top of the screen
  2. Click desired assignment to view reviews from
  3. Click on "Your scores" button
  4. Scroll down past assignment score heatgrid
  5. You will see the reviews that others have made about you if the instructor has enabled that setting on this assignment

If you are logged in as an instructor:

  1. Click "Manage..." then "Assignments" at the top of the screen
  2. Click on the "view scores" button on the desired assignment
  3. Click the plus next to the team that you are interested in
  4. Click on the "Teammate Reviews" tab at the top of the panel
  5. Select the student that you wish to see the reviews about
  6. The relevant heatgrid will display with a descriptive title

Test Login Credentials

Instructor

  • username: instructor6
  • password: password

Student

  • username: student7185
  • password: password

Added Unit Tests

Added System Tests

RSpec tests will be added as a component of the first code submission.

Added System Tests

The following tests were conducted as follows:
- Sign in as test student7185 above
- Navigate to assignments
- Click on the Final Project assignment
- Click on "Your scores" button

1) View student reviews received from teammates and given to teammates from a student view

---Code still being developed---

2) View teammate reviews as a student when enabled

The teammate reviews will be shown at the bottom of the screen because it has been enabled by the instructor:

Student view shows teammate reviews of student if allowed
Student view shows teammate reviews of student if allowed

3) View composite score of reviews from a student view

---Code still being developed---

The following tests were conducted as follows:
- Sign in as instructor above
- Click Manage -> Assignments
- Click on the View Scores of Final Project assignment
- Click into any team

1) View student reviews received from teammates and given to teammates from an instructor view

---Code still being developed---

2) View composite score of reviews from an instructor view

---Code still being developed---

3) Show descriptive name for heatmaps from an instructor view

The displayed heatgrid has a new, descriptive title describing exactly what reviews are being displayed:

Teammate Review Heatmap has descriptive title
Teammate Review Heatmap has descriptive title

Manual Testing for Anonymized User View:

The following tests were conducted as follows:
- Sign in as test student7185 above
- Navigate to assignments
- Click on the Final Project assignment
- Click on "Your scores" button

1) View your teammates names in the Team members section of review scores

2) Switch to Anonymized View and view your teammates names in the Team members section of review scores