Expertiza_Wiki - User contributions [en]

CSC/ECE 517 Spring 2021 - E2113. Enhancements to review grader

2021-03-31T20:53:39Z

Nsundar:

==Background==

When the staff grade reviews, they are viewed in the review-grader interface. The staff can assign the score and provide feedback using the textboxes provided in the right columns of the table. The staff are also allowed to read the reviews in the summary view.

There appears to be certain sections of the main review-grader page as well as the summary page that are not working correctly or there is a desire for them to be updated. These issues have to do with calculation of a student's score / average score, the visual representation of the scores, efficient usage of space within the view, and a need for rows of the table(s) to be numbered. We go into further depth about the issues later in this document.

==Problem Statement==
Certain aspects of the UI and minor score calculation logic of the review grader system in Expertiza need to be enhanced/refactored. These issues are not major and mainly have to do with UI, and therefore should not require the development of new unit tests. Below is a detailed description of actionable issues that will be addressed in this work.

===The Main Review-Grader Page===
====Issue Description====
#Fix the “Score awarded/average score” column so that it is populated with the correct numbers. It is supposed to report the score by the current student reviewer in the first round, the average score by all reviewers in the first round, and ditto for the second round. If the number of rounds ≠ 2, then the number of scores should be adjusted appropriately.
#Display the scores in the metrics column as a bar graph. Precautions must to be taken to not make any column wider, since the page needs to display a lot of information horizontally
#In the view shown, “Reviewer”, “Reviews done”, and “Team reviewed” are too wide, whereas “Assign grade and write comments” is too narrow. Adjustments to column widths, thus needs to be done
#Number the rows of the table (e.g., “2. Student 8370”) so it is easy to count the lines. This will help assign each TA (and the instructor) an equal number of reviews to grade.

====Design Strategy====
#The width of column is set up in the _review_report.html.erb file from line 37 to line 43. Change the percentage distributed to every column to adjust the width
#In the _review_report.html.erb file, Line 61, we need to replace the 'each' method with 'each_with_index' to make sure that all rows are indexed. In line 69 add a line to display the index number and make sure that the value of the variable must be index+1, since the values are zero indexed

====Design Pattern====
This project mainly focuses on the UI, so no design pattern is needed.

====Test Plan====
#Login using an instructor account, such as: instructor6 / password
#Go to "Manage >> Assignments" and choose one assignment with reviews, i.e., "Program 1", and click on "View Reports".
#There will be a dropdown selector to choose which report to view - select "Review report" from the dropdown, and click "View".
#''Check if the reviewer names and team names are all anonymized.''
#''Check if the checkbox items are displayed as expected.''
#''Check if column widths are appropriate..''
#''Check if “Review: Round1” is changed to “Review Round 1”.''

===The Summary Page===
====Issue Description====

CSC/ECE 517 Spring 2021 - E2113. Enhancements to review grader

2021-03-31T20:24:20Z

Nsundar:

==Background==

When the staff grade reviews, they are viewed in the review-grader interface. The staff can assign the score and provide feedback using the textboxes provided in the right columns of the table. The staff are also allowed to read the reviews in the summary view.

There appears to be certain sections of the main review-grader page as well as the summary page that are not working correctly or there is a desire for them to be updated. These issues have to do with calculation of a student's score / average score, the visual representation of the scores, efficient usage of space within the view, and a need for rows of the table(s) to be numbered. We go into further depth about the issues later in this document.

==Problem Statement==
Certain aspects of the UI and minor score calculation logic of the review grader system in Expertiza need to be enhanced/refactored. These issues are not major and mainly have to do with UI, and therefore should not require the development of new unit tests. Below is a detailed description of actionable issues that will be addressed in this work.

===The Main Review-Grader Page===
====Issue Description====
#Fix the “Score awarded/average score” column so that it is populated with the correct numbers. It is supposed to report the score by the current student reviewer in the first round, the average score by all reviewers in the first round, and ditto for the second round. If the number of rounds ≠ 2, then the number of scores should be adjusted appropriately.
#Display the scores in the metrics column as a bar graph. Precautions must to be taken to not make any column wider, since the page needs to display a lot of information horizontally
====Test Plan====
#Login using an instructor account, such as: instructor6 / password
#Go to "Manage >> Assignments" and choose one assignment with reviews, i.e., "Program 1", and click on "View Reports".
#There will be a dropdown selector to choose which report to view - select "Review report" from the dropdown, and click "View".
#
#''Check if the reviewer names and team names are all anonymized.''
#''Check if the checkbox items are displayed as expected.''
#''Check if column widths are appropriate..''
#''Check if “Review: Round1” is changed to “Review Round 1”.''

===The Summary Page===
====Issue Description====

File:Expertiza capture.png

2021-03-27T03:47:42Z

Nsundar: Nsundar uploaded a new version of File:Expertiza capture.png

File:Expertiza capture.png

2021-03-27T03:46:53Z

Nsundar:

File:Expertiza capture.PNG

2021-03-27T03:43:41Z

Nsundar:

CSC/ECE 517 Spring 2021 - E2106. Fix view in student task/list page

2021-03-27T03:43:10Z

Nsundar:

This page details project documentation for the CSC/ECE 517 Spring 2021, "E2106 Fix view in student task/list page" project.

== Background ==

The student_task/list page is the page displayed to a student after logging in into expertiza. It has mainly two div(s), one to show the upcoming tasks along with some important information, another div is to display all the assignments and their information in which the student is participating. We have to modify the view to remove extraneous information and make it clearer and more concise.

=== Mentor ===

Saurabh Shingte, svshingt@ncsu.edu

=== Team Members ===

* Sharath Kumar (svkumar2@ncsu.edu)
* Nikhil Sundaraswamy (nsundar@ncsu.edu)

== Issues ==
* Modify the view to remove extraneous information and make it clearer and more concise
* Add checkbox to Publishing Rights column, when checked the permission_granted boolean variable is set to true
* In general, no column needs to be shown if there are no values in it. This would apply to “Badges” if no assignments in the list contain badges.
* Also, it doesn’t make sense to have a column for Review Grade, and not for the grade for submitted work (“Submission Grade”).
* We should consider the cases where a student might be enrolled in assignments from more than one course. In this case, assignments should be grouped by course, with the course name given before the listing of the first assignment. (This allows for the column for course to be removed, making the display more compact.

== Functionality Testing ==
LogIn to Expertiza as any of the students mentioned below. Without the changes the original view is as shown below.

*instructor6

* student7487

* student7490

* student7496

Password for logging in for any student/instructor is : password

Once logged into the page click on "Assignments" tab. A modified view is observed as compared to the earlier view

[[File:student_task_original_view.png]]

To address issue 1 we added CSS style properties and removed the unnecessary space between two divs by using HTML comment.

[[File:changes_in_list_html_erb_file.png]]

The modified view is as shown below

[[File:student_task_modified_view.png]]

To address issue 2, we did the following modifications

* In app/assests/javascripts/submissions.js, we have added an ajax call to update the permission_granted boolean variable in Participant table

[[File:submission_js_new.png]]

* In app/controller/student_task_controller.rb, we are defining a method named permission_granted which routes the action from checkbox event

[[File:student_task_controller.png]]

* app/views/student_task/list.html.erb, we have added code to include an information banner on Publishing Rights column and checkboxes

[[File:publishing_rights_info_banner.png]]

* In config/routes.rb, we have added a functionality to route the checkbox event to permission_granted action in student_task_controller.rb

[[File:routes_rb.png]]

After all proposed modifications, we see the below shown view

[[File:expertiza_capture.jpg]]

== Test Plan ==

Since this project involved mostly UI changes it was not necessary to write test cases for the same. So there is '''no test plan'''

== Modified Files ==
*

== Links ==
* https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Spring_2021_-_E2106._Fix_view_in_student_task/list_page

* https://github.com/sharathKV/expertiza

* http://152.7.99.46:8080/

* https://github.com/expertiza/expertiza/pull/1903

CSC/ECE 517 Spring 2021 - E2106. Fix view in student task/list page

2021-03-27T03:41:32Z

Nsundar:

This page details project documentation for the CSC/ECE 517 Spring 2021, "E2106 Fix view in student task/list page" project.

== Background ==

The student_task/list page is the page displayed to a student after logging in into expertiza. It has mainly two div(s), one to show the upcoming tasks along with some important information, another div is to display all the assignments and their information in which the student is participating. We have to modify the view to remove extraneous information and make it clearer and more concise.

=== Mentor ===

Saurabh Shingte, svshingt@ncsu.edu

=== Team Members ===

* Sharath Kumar (svkumar2@ncsu.edu)
* Nikhil Sundaraswamy (nsundar@ncsu.edu)

== Issues ==
* Modify the view to remove extraneous information and make it clearer and more concise
* Add checkbox to Publishing Rights column, when checked the permission_granted boolean variable is set to true
* In general, no column needs to be shown if there are no values in it. This would apply to “Badges” if no assignments in the list contain badges.
* Also, it doesn’t make sense to have a column for Review Grade, and not for the grade for submitted work (“Submission Grade”).
* We should consider the cases where a student might be enrolled in assignments from more than one course. In this case, assignments should be grouped by course, with the course name given before the listing of the first assignment. (This allows for the column for course to be removed, making the display more compact.

== Functionality Testing ==
LogIn to Expertiza as any of the students mentioned below. Without the changes the original view is as shown below.

*instructor6

* student7487

* student7490

* student7496

Password for logging in for any student/instructor is : password

Once logged into the page click on "Assignments" tab. A modified view is observed as compared to the earlier view

[[File:student_task_original_view.png]]

To address issue 1 we added CSS style properties and removed the unnecessary space between two divs by using HTML comment.

[[File:changes_in_list_html_erb_file.png]]

The modified view is as shown below

[[File:student_task_modified_view.png]]

To address issue 2, we did the following modifications

* In app/assests/javascripts/submissions.js, we have added an ajax call to update the permission_granted boolean variable in Participant table

[[File:submission_js_new.png]]

* In app/controller/student_task_controller.rb, we are defining a method named permission_granted which routes the action from checkbox event

[[File:student_task_controller.png]]

* app/views/student_task/list.html.erb, we have added code to include an information banner on Publishing Rights column and checkboxes

[[File:publishing_rights_info_banner.png]]

* In config/routes.rb, we have added a functionality to route the checkbox event to permission_granted action in student_task_controller.rb

[[File:routes_rb.png]]

After all proposed modifications, we see the below shown view

== Test Plan ==

Since this project involved mostly UI changes it was not necessary to write test cases for the same. So there is '''no test plan'''

== Modified Files ==
*

== Links ==
* https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Spring_2021_-_E2106._Fix_view_in_student_task/list_page

* https://github.com/sharathKV/expertiza

* http://152.7.99.46:8080/

* https://github.com/expertiza/expertiza/pull/1903

CSC/ECE 517 Spring 2021 - E2106. Fix view in student task/list page

2021-03-27T03:40:07Z

Nsundar:

This page details project documentation for the CSC/ECE 517 Spring 2021, "E2106 Fix view in student task/list page" project.

== Background ==

The student_task/list page is the page displayed to a student after logging in into expertiza. It has mainly two div(s), one to show the upcoming tasks along with some important information, another div is to display all the assignments and their information in which the student is participating. We have to modify the view to remove extraneous information and make it clearer and more concise.

=== Mentor ===

Saurabh Shingte, svshingt@ncsu.edu

=== Team Members ===

* Sharath Kumar (svkumar2@ncsu.edu)
* Nikhil Sundaraswamy (nsundar@ncsu.edu)

== Issues ==
* Modify the view to remove extraneous information and make it clearer and more concise
* Add checkbox to Publishing Rights column, when checked the permission_granted boolean variable is set to true
* In general, no column needs to be shown if there are no values in it. This would apply to “Badges” if no assignments in the list contain badges.
* Also, it doesn’t make sense to have a column for Review Grade, and not for the grade for submitted work (“Submission Grade”).
* We should consider the cases where a student might be enrolled in assignments from more than one course. In this case, assignments should be grouped by course, with the course name given before the listing of the first assignment. (This allows for the column for course to be removed, making the display more compact.

== Functionality Testing ==
LogIn to Expertiza as any of the students mentioned below. Without the changes the original view is as shown below.

*instructor6

* student7487

* student7490

* student7496

Password for logging in for any student/instructor is : password

Once logged into the page click on "Assignments" tab. A modified view is observed as compared to the earlier view

[[File:student_task_original_view.png]]

To address issue 1 we added CSS style properties and removed the unnecessary space between two divs by using HTML comment.

[[File:changes_in_list_html_erb_file.png]]

The modified view is as shown below

[[File:student_task_modified_view.png]]

To address issue 2, we did the following modifications

* In app/assests/javascripts/submissions.js, we have added an ajax call to update the permission_granted boolean variable in Participant table

[[File:submission_js_new.png]]

* In app/controller/student_task_controller.rb, we are defining a method named permission_granted which routes the action from checkbox event

[[File:student_task_controller.png]]

* app/views/student_task/list.html.erb, we have added code to include an information banner on Publishing Rights column and checkboxes

[[File:publishing_rights_info_banner.png]]

* In config/routes.rb, we have added a functionality to route the checkbox event to permission_granted action in student_task_controller.rb

[[File:routes_rb.png]]

== Test Plan ==

Since this project involved mostly UI changes it was not necessary to write test cases for the same. So there is '''no test plan'''

== Modified Files ==
*

== Links ==
* https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Spring_2021_-_E2106._Fix_view_in_student_task/list_page

* https://github.com/sharathKV/expertiza

* http://152.7.99.46:8080/

* https://github.com/expertiza/expertiza/pull/1903

CSC/ECE 517 Spring 2021 - E2106. Fix view in student task/list page

2021-03-27T03:31:15Z

Nsundar:

This page details project documentation for the CSC/ECE 517 Spring 2021, "E2106 Fix view in student task/list page" project.

== Background ==

The student_task/list page is the page displayed to a student after logging in into expertiza. It has mainly two div(s), one to show the upcoming tasks along with some important information, another div is to display all the assignments and their information in which the student is participating. We have to modify the view to remove extraneous information and make it clearer and more concise.

=== Mentor ===

Saurabh Shingte, svshingt@ncsu.edu

=== Team Members ===

* Sharath Kumar (svkumar2@ncsu.edu)
* Nikhil Sundaraswamy (nsundar@ncsu.edu)

== Issues ==
* Modify the view to remove extraneous information and make it clearer and more concise
* Add checkbox to Publishing Rights column, when checked the permission_granted boolean variable is set to true
* In general, no column needs to be shown if there are no values in it. This would apply to “Badges” if no assignments in the list contain badges.
* Also, it doesn’t make sense to have a column for Review Grade, and not for the grade for submitted work (“Submission Grade”).
* We should consider the cases where a student might be enrolled in assignments from more than one course. In this case, assignments should be grouped by course, with the course name given before the listing of the first assignment. (This allows for the column for course to be removed, making the display more compact.

== Functionality Testing ==
LogIn to Expertiza as any of the students mentioned below. Without the changes the original view is as shown below.

*instructor6

* student7487

* student7490

* student7496

Password for logging in for any student/instructor is : password

[[File:student_task_original_view.png]]

To address issue 1 we added CSS style properties and removed the unnecessary space between two divs by using HTML comment.

[[File:changes_in_list_html_erb_file.png]]

The modified view is as shown below

[[File:student_task_modified_view.png]]

To address issue 2, we did the following modifications

* In app/assests/javascripts/submissions.js, we have added an ajax call to update the permission_granted boolean variable in Participant table

[[File:submission_js_new.png]]

* In app/controller/student_task_controller.rb, we are defining a method named permission_granted which routes the action from checkbox event

[[File:student_task_controller.png]]

* app/views/student_task/list.html.erb, we have added code to include an information banner on Publishing Rights column and checkboxes

[[File:publishing_rights_info_banner.png]]

* In config/routes.rb, we have added a functionality to route the checkbox event to permission_granted action in student_task_controller.rb

[[File:routes_rb.png]]

== Links ==
* https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Spring_2021_-_E2106._Fix_view_in_student_task/list_page

* https://github.com/sharathKV/expertiza

* http://152.7.99.46:8080/

* https://github.com/expertiza/expertiza/pull/1903