CSC/ECE 517 Fall 2016 E1696 Improve Self-Review: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
No edit summary
 
(15 intermediate revisions by one other user not shown)
Line 8: Line 8:
== Task Description ==
== Task Description ==


Display self-review scores juxtaposed with peer-review scores in the regular “View Scores” page and the alternate (heat-map) view.
1) Identifying the Expertiza project pages which takes time to load using Rack-mini-profiler and Flamegraphs.
2) Propose  fixes which would improve the Expertiza project
3) Optimizing the view ,models and controllers for few of these corresponding fixes to improve the load time of these pages


Combine self-review and peer-review scores to derive a composite score, based on closeness to the peer-review.


Author(s) should be required to submit their self-evaluation(s) before seeing the results of their peer evaluations.


== Definitions (in context of the project) ==
== Gems Installed  ==


*Peer-review : Review of your work done by others in the class.
*rack-mini-profiler :- Middleware that displays speed badge for every html page. Designed to work both in production and in development
*Self-review : Review of your work done by you. This is unique to each individual (and not each team).
*flamegraphs :- Flame graphs are a visualization of profiled software, allowing the most frequent code-paths to be identified quickly and accurately.
*Author : The one that writes the review. Could be a peer or yourself.
*stackprof:-A sampling call-stack profiler for ruby 2.1+.Downloaded as a dependency for rack-mini-profiler.
*Instructor : The admin of the system, ideally, the professor and the TAs.
*fast_stack :-fast_stack is dynamically resizable data structure optimized for fast iteration over the large arrays of similar elements avoiding memory fragmentation.
*Instructor-score : Score assigned by the instructor or his TAs.


== Edge Cases and Constraints ==
== Edge Cases and Constraints ==
Line 37: Line 36:
=== MVC Pattern ===
=== MVC Pattern ===


Expertiza is developed using the nuances of the MVC Design Architecture where Models, Views and Controllers are separated
Expertiza is developed using the nuances of the MVC Design Architecture where Models, Views and Controllers are coupled to a very low level. The model manages all the data and the logic related to the data. The controller accepts data from the Models and feeds the Views with a favorable form of the data. The view presents the data to the user.


=== Observer Pattern ===
The advantage of using the Observer Pattern is that the object being observed need not worry about any changes in its state. An Observer Class is added whose objects will look into the changes in the state of the Object being observed.
Here, the observers are the reviewers and the assignment is the object being observed. In case of Self-Review, an Observer can be used to tell the system that the reviewer is now open to reviewing the works of his peers.


[[File:Design.jpg]]
[[File:Design.jpg]]
[[File:Design2.jpg]]




== Implementation ==
== Implementation ==


=== Current Implementation ===
=== Devising a formula for Self-Review scores ===
 
Changes made in app/views/grades/_participant.html.erb, app/views/grades/_participant_charts.html.erb and app/views/grades/_participant_title.html.erb.
 
Here and everywhere else in this document, 'Raw Self-Review' score indicates the score assigned by the participants assessing one's own work. Self-Review score is a metric used to measure the extent of agreement between the Raw Self-Review score and the Average Peer-Review score.
 
The highlighted portion of the screenshot below shows how the Self-Review score is obtained from the Raw Self-Review score.
 
[[File:mypic5.jpg]]
 
=== Graphically representing Self-Review score ===
 
Changes made in app/views/grades/_participant_charts.html.erb.
 
In accordance with the design paradigm of Expertiza, we developed a chart (or a circle) to represent the Self-Review Score (Again, remember that this is not the Raw Self-Review score. The Raw Self-Review score is of no use to the participants because it was, in fact, their opinion of the work done. The computed Self-Review score, on the other hand, is more useful as it depicts the level of agreement between the opinions of the participant and the reviewers.). We also ensured that the circle is colored similar to the other charts in the page (other than the final score chart, which was deliberately designed to be bright and unique).


Self-Review is partially functional in the current build of Expertiza. The file _self_review.html.erb in submitted_content view provides the links to 'View', 'Update', 'Edit' and 'Begin' self-reviews. Then the control transfers control to view() method in response controller.  
The following screenshot of a code snippet shows the creation of the circle for Self-Review score.


=== Juxtaposing Peer and Self-Review Scores ===
[[File:mypic2.jpg]]


The system currently uses view_my_score() method in grades_controller to display the scores in peer-review. The same set of statements used in view_my_score() can also be used to display the self-review score.
=== Consolidation of Self-Review score into Final score ===


=== Combine Self and Peer-Reviews into a Composite Score ===
Changes made in app/views/grades/_participant.html.erb, app/views/grades/_participant_charts.html.erb and app/views/grades/_participant_title.html.erb.


For starters, we are going to use a simple formula to determine the Self-Review score.
Self-Review score cannot be a major part of the Final score. On the other hand, the Peer-Review scores must be the dominant component of Final score. After consulting with the Professor, we decided to give a 10% weightage to the Self-Review score and a 90% weightage to the Peer-Review score. We also deliberately made this non-configurable in the future because it is not to be meddled with; a small mistake in coding can lead to repercussions in different controllers that is undesirable.


Composite Score = 100 - ( | Self-Review Score - Average Peer-Review Score | )
The highlighted portion of the following screenshot shows the composition of the Final score.


=== Complete Self-Review before checking Peer-Review Scores ===
[[File:mypic3.jpg]]


The inclusion of a check_self_review_status() method in grades controller can be used to determine the presence of an already completed self-review. It returns false in the absence of an already completed self-review, otherwise it returns true.
=== Ensuring Self-Review is done before Viewing scores ===
 
Changes made in app/views/student_task/view.html.erb.
 
One sensible requirement of the project was to ensure every participant of an assignment evaluates one's own work before looking into the scores assigned by others. This is done to avoid an explicit or implicit comparison of the raw self-review score to the peer-review scores.
 
By ensuring that every participant performs the self-review before looking into the scores assigned to his/her work, we also ensure that the raw self-review score is an honest indication of what the participant thinks about one's own work.
 
In the screenshot below, the 'if' and 'else' statements ensure the same.
 
[[File:mypic4.jpg]]


== Testing ==
== Testing ==


# Juxtaposing Peer and Self-Review Scores
Kindly check the Screencast video.
From the Assignment Home Page, click on 'Your Scores' to view both Average Peer and Self-Review Scores.
 
# Juxtaposing Peer and Self-Review Scores - From the Assignment Home Page, click on 'Your Scores' to view both Average Peer-Review, Self-Review and Final Scores.
# Combine Self and Peer-Reviews into a Composite Score - From the Assignment Home Page, click on 'Your Scores' to view the Composite Score assigned to you based on the closeness of your Self-Review score with the Average Peer-Review Score.  
# Combine Self and Peer-Reviews into a Composite Score - From the Assignment Home Page, click on 'Your Scores' to view the Composite Score assigned to you based on the closeness of your Self-Review score with the Average Peer-Review Score.  
# Complete Self-Review before checking Peer-Review Scores - From the Assignment Home Page, click on 'Others' Work' to try to submit Peer-Reviews for the projects submitted by others. However, this should be forbidden unless you have submitted a Self-Review of your project.
# Complete Self-Review before checking Peer-Review Scores - From the Assignment Home Page, click on 'Others' Work' to try to submit Peer-Reviews for the projects submitted by others. However, this should be forbidden unless you have submitted a Self-Review of your project.
== Scope for Future Work ==
*The words used in the self-reviews can be used to judge the seriousness of the reviews.
*The changes in the scores of the self-reviews can be used to judge whether teams have worked to their own expectations.
*In order to establish a common standard in the reviewing system, the absolute change in the review scores can be used to judge an individual.

Latest revision as of 22:43, 3 November 2017

IMPROVE SELF-REVIEW

Introduction

Expertiza is an online system that is used by students to view/submit assignments and review others' work. Expertiza also provides tools to visualize the scores and gauge the improvements made during the course semester. It also facilitates and monitors team projects. It is targeted at educational and non-profit organizations. The project is funded by the National Software Foundation (NSF), NCSU Learning in a Technology-Rich Environment (LITRE) program, the NCSU Faculty Center for Teaching and Learning, the NCSU STEM Initiative, and the Center for Advanced Computing and Communication.

Expertiza is an open-source project with the source code available as a public repository on GitHub. It is developed using Ruby on Rails and is increasingly becoming robust thanks to the innumerable bugs being fixed by the community. The project has a micro-blog on SourceForge where the developer community report bugs and document updates.

Task Description

1) Identifying the Expertiza project pages which takes time to load using Rack-mini-profiler and Flamegraphs. 2) Propose fixes which would improve the Expertiza project 3) Optimizing the view ,models and controllers for few of these corresponding fixes to improve the load time of these pages


Gems Installed

  • rack-mini-profiler :- Middleware that displays speed badge for every html page. Designed to work both in production and in development
  • flamegraphs :- Flame graphs are a visualization of profiled software, allowing the most frequent code-paths to be identified quickly and accurately.
  • stackprof:-A sampling call-stack profiler for ruby 2.1+.Downloaded as a dependency for rack-mini-profiler.
  • fast_stack :-fast_stack is dynamically resizable data structure optimized for fast iteration over the large arrays of similar elements avoiding memory fragmentation.

Edge Cases and Constraints

  1. One can view the peer-reviews done only upon the completion of his/her self-review.
  2. The final score of the self-review depends on the agreement of the self-review scores with peer-review scores.
  3. While comparing the self-review scores with peer-review scores, more weight must be given to the instructor-score.
  4. While considering the reviews done by others in the class, the average must be used.
  5. There is no feedback for self-reviews.
  6. Self-reviews are done individually and not as a team.
  7. The self-review scores of each member of a team does not affect the self-review scores of the other members in the team.

Approach

Student has the task of performing review of others' work as well as self review. Every team gets their review from peers and at present the composite score is the average of all the review scores that they have received. Added to this score, the self review score is also calculated for each team member, based on its closeness to the peer reviewed score. To implement this, the student needs to review his own work based on the same rubrics used for peer review and only after completion of the self review, their peer review scores will be visible. The composite score is calculated by the sum of the peer review average score and 100 minus the difference between self and peer review score. The self review score is displayed adjacent to the peer review score. As the composite score is based on the individual self reviews, they vary for each member of the team unlike the peer review score which is the same.

MVC Pattern

Expertiza is developed using the nuances of the MVC Design Architecture where Models, Views and Controllers are coupled to a very low level. The model manages all the data and the logic related to the data. The controller accepts data from the Models and feeds the Views with a favorable form of the data. The view presents the data to the user.

Observer Pattern

The advantage of using the Observer Pattern is that the object being observed need not worry about any changes in its state. An Observer Class is added whose objects will look into the changes in the state of the Object being observed.

Here, the observers are the reviewers and the assignment is the object being observed. In case of Self-Review, an Observer can be used to tell the system that the reviewer is now open to reviewing the works of his peers.


Implementation

Devising a formula for Self-Review scores

Changes made in app/views/grades/_participant.html.erb, app/views/grades/_participant_charts.html.erb and app/views/grades/_participant_title.html.erb.

Here and everywhere else in this document, 'Raw Self-Review' score indicates the score assigned by the participants assessing one's own work. Self-Review score is a metric used to measure the extent of agreement between the Raw Self-Review score and the Average Peer-Review score.

The highlighted portion of the screenshot below shows how the Self-Review score is obtained from the Raw Self-Review score.

Graphically representing Self-Review score

Changes made in app/views/grades/_participant_charts.html.erb.

In accordance with the design paradigm of Expertiza, we developed a chart (or a circle) to represent the Self-Review Score (Again, remember that this is not the Raw Self-Review score. The Raw Self-Review score is of no use to the participants because it was, in fact, their opinion of the work done. The computed Self-Review score, on the other hand, is more useful as it depicts the level of agreement between the opinions of the participant and the reviewers.). We also ensured that the circle is colored similar to the other charts in the page (other than the final score chart, which was deliberately designed to be bright and unique).

The following screenshot of a code snippet shows the creation of the circle for Self-Review score.

Consolidation of Self-Review score into Final score

Changes made in app/views/grades/_participant.html.erb, app/views/grades/_participant_charts.html.erb and app/views/grades/_participant_title.html.erb.

Self-Review score cannot be a major part of the Final score. On the other hand, the Peer-Review scores must be the dominant component of Final score. After consulting with the Professor, we decided to give a 10% weightage to the Self-Review score and a 90% weightage to the Peer-Review score. We also deliberately made this non-configurable in the future because it is not to be meddled with; a small mistake in coding can lead to repercussions in different controllers that is undesirable.

The highlighted portion of the following screenshot shows the composition of the Final score.

Ensuring Self-Review is done before Viewing scores

Changes made in app/views/student_task/view.html.erb.

One sensible requirement of the project was to ensure every participant of an assignment evaluates one's own work before looking into the scores assigned by others. This is done to avoid an explicit or implicit comparison of the raw self-review score to the peer-review scores.

By ensuring that every participant performs the self-review before looking into the scores assigned to his/her work, we also ensure that the raw self-review score is an honest indication of what the participant thinks about one's own work.

In the screenshot below, the 'if' and 'else' statements ensure the same.

Testing

Kindly check the Screencast video.

  1. Juxtaposing Peer and Self-Review Scores - From the Assignment Home Page, click on 'Your Scores' to view both Average Peer-Review, Self-Review and Final Scores.
  2. Combine Self and Peer-Reviews into a Composite Score - From the Assignment Home Page, click on 'Your Scores' to view the Composite Score assigned to you based on the closeness of your Self-Review score with the Average Peer-Review Score.
  3. Complete Self-Review before checking Peer-Review Scores - From the Assignment Home Page, click on 'Others' Work' to try to submit Peer-Reviews for the projects submitted by others. However, this should be forbidden unless you have submitted a Self-Review of your project.

Scope for Future Work

  • The words used in the self-reviews can be used to judge the seriousness of the reviews.
  • The changes in the scores of the self-reviews can be used to judge whether teams have worked to their own expectations.
  • In order to establish a common standard in the reviewing system, the absolute change in the review scores can be used to judge an individual.