Expertiza_Wiki - User contributions [en]

IndependentStudy Visualizations

2019-12-04T00:38:48Z

Sseelam2:

===Introduction===

This project fixes the way graphs are represented in the review report. Expertiza interprets metrics in the form of a graph in the review report. This Project also refractors
review_mapping_helper.rb.

===Problem Statement===
* Currently, the bars in the graphs overlap and they are not positioned properly.
* The legends (vol. and avg vol.) are present in every cell in the metrics column.
* The current code assumes that there are no more than three rounds.
* Refactored review_mapping_helper.rb.
* Add comments to methods.

===Files modified===
app/assets/stylesheets/legend.css
app/helpers/charts_helper.rb
app/helpers/review_mapping_helper.rb
app/views/reports/_calibration_report.html.erb
app/views/reports/_feedback_report.html.erb
app/views/reports/_review_report.html.erb
app/views/reports/_team_score.html.erb

===Solution===
1. Currently, the bars in the graphs overlap and they are not positioned properly and the legends (vol. and avg vol.) are present in every cell in the metrics column.
This change is present in the files - app/views/reports/_review_report.html.erb and app/helpers/review_mapping_helper.rb

'''Before'''
 
[[File:Before_Graph.jpg]]
 

'''After'''
 
[[File:After_Graph.jpg]]
 
2. The current code assumes that there are no more than three rounds. Changes can be found in app/helpers/review_mapping_helper.rb

'''Before'''
def initialize_chart_elements(reviewer)
round = 0
labels = []
reviewer_data = []
all_reviewers_data = []
if @all_reviewers_avg_vol_in_round_1 > 0
round += 1
labels.push '1st'
reviewer_data.push reviewer.avg_vol_in_round_1
all_reviewers_data.push @all_reviewers_avg_vol_in_round_1
end
if @all_reviewers_avg_vol_in_round_2 > 0
round += 1
labels.push '2nd'
reviewer_data.push reviewer.avg_vol_in_round_2
all_reviewers_data.push @all_reviewers_avg_vol_in_round_2
end
if @all_reviewers_avg_vol_in_round_3 > 0
round += 1
labels.push '3rd'
reviewer_data.push reviewer.avg_vol_in_round_3
all_reviewers_data.push @all_reviewers_avg_vol_in_round_3
end
labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

'''After'''

def initialize_chart_elements(reviewer)
#round = 0
labels = []
reviewer_data = []
all_reviewers_data = []

(1..@assignment.num_review_rounds).each do |round_num|
#round += 1
labels.push round_num.to_s
reviewer_data.push eval("reviewer.avg_vol_in_round_#{round_num}")
all_reviewers_data.push instance_variable_get("@all_reviewers_avg_vol_in_round_#{round_num}")
end
labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

3. Refractored review_mapping_helper.rb.
Refactored review_mapping_helper.rb. Few of the changes are reducing the length of method names, adding comments, removing redundant code,
adding string interpolation.

please see the pull request for detailed code. [https://github.com/expertiza/expertiza/pull/1493]

IndependentStudy Visualizations

2019-12-04T00:36:41Z

Sseelam2:

===Introduction===

This project fixes the way graphs are represented in the review report. Expertiza interprets metrics in the form of a graph in the review report. This Project also refractors
review_mapping_helper.rb.

===Problem Statement===
Currently, the bars in the graphs overlap and they are not positioned properly.
The legends (vol. and avg vol.) are present in every cell in the metrics column.
The current code assumes that there are no more than three rounds.
Removed charts_helpers.rb since it was not being used anywhere in the code.
Refractored review_mapping_helper.rb.
Add comments to methods.

===Files modified===
app/assets/stylesheets/legend.css
app/helpers/charts_helper.rb
app/helpers/review_mapping_helper.rb
app/views/reports/_calibration_report.html.erb
app/views/reports/_feedback_report.html.erb
app/views/reports/_review_report.html.erb
app/views/reports/_team_score.html.erb

===Solution===
1. Currently, the bars in the graphs overlap and they are not positioned properly and the legends (vol. and avg vol.) are present in every cell in the metrics column.
This change is present in the files - app/views/reports/_review_report.html.erb and app/helpers/review_mapping_helper.rb

'''Before'''
 
[[File:Before_Graph.jpg]]
 

'''After'''
 
[[File:After_Graph.jpg]]
 
2. The current code assumes that there are no more than three rounds. Changes can be found in app/helpers/review_mapping_helper.rb

'''Before'''
def initialize_chart_elements(reviewer)
round = 0
labels = []
reviewer_data = []
all_reviewers_data = []
if @all_reviewers_avg_vol_in_round_1 > 0
round += 1
labels.push '1st'
reviewer_data.push reviewer.avg_vol_in_round_1
all_reviewers_data.push @all_reviewers_avg_vol_in_round_1
end
if @all_reviewers_avg_vol_in_round_2 > 0
round += 1
labels.push '2nd'
reviewer_data.push reviewer.avg_vol_in_round_2
all_reviewers_data.push @all_reviewers_avg_vol_in_round_2
end
if @all_reviewers_avg_vol_in_round_3 > 0
round += 1
labels.push '3rd'
reviewer_data.push reviewer.avg_vol_in_round_3
all_reviewers_data.push @all_reviewers_avg_vol_in_round_3
end
labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

'''After'''

def initialize_chart_elements(reviewer)
#round = 0
labels = []
reviewer_data = []
all_reviewers_data = []

(1..@assignment.num_review_rounds).each do |round_num|
#round += 1
labels.push round_num.to_s
reviewer_data.push eval("reviewer.avg_vol_in_round_#{round_num}")
all_reviewers_data.push instance_variable_get("@all_reviewers_avg_vol_in_round_#{round_num}")
end
labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

3. Refractored review_mapping_helper.rb.
Refactored review_mapping_helper.rb. Few of the changes are reducing the length of method names, adding comments, removing redundant code,
adding string interpolation.

please see the pull request for detailed code. [https://github.com/expertiza/expertiza/pull/1493]

File:After Graph.jpg

2019-12-04T00:28:14Z

Sseelam2:

IndependentStudy Visualizations

2019-12-04T00:25:59Z

Sseelam2:

'''Introduction:'''
This project fixes the way graphs are represented in the review report. Expertiza interprets metrics in the form of a graph in the review report. This Project also refractors
review_mapping_helper.rb.

'''Problem Statement:'''
Currently, the bars in the graphs overlap and they are not positioned properly.
The legends (vol. and avg vol.) are present in every cell in the metrics column.
The current code assumes that there are no more than three rounds.
Removed charts_helpers.rb since it was not being used anywhere in the code.
Refractored review_mapping_helper.rb.
Add comments to methods.
'''
Files modified:'''
app/assets/stylesheets/legend.css
app/helpers/charts_helper.rb
app/helpers/review_mapping_helper.rb
app/views/reports/_calibration_report.html.erb
app/views/reports/_feedback_report.html.erb
app/views/reports/_review_report.html.erb
app/views/reports/_team_score.html.erb

'''Solution:'''
1. Currently, the bars in the graphs overlap and they are not positioned properly and the legends (vol. and avg vol.) are present in every cell in the metrics column.
This change is present in the files - app/views/reports/_review_report.html.erb and app/helpers/review_mapping_helper.rb

Before:
[[File:Before_Graph.jpg]]

After:
[[File:After_Graph.jpg]]

2. The current code assumes that there are no more than three rounds. Changes can be found in app/helpers/review_mapping_helper.rb

Before:
def initialize_chart_elements(reviewer)
round = 0
labels = []
reviewer_data = []
all_reviewers_data = []
if @all_reviewers_avg_vol_in_round_1 > 0
round += 1
labels.push '1st'
reviewer_data.push reviewer.avg_vol_in_round_1
all_reviewers_data.push @all_reviewers_avg_vol_in_round_1
end
if @all_reviewers_avg_vol_in_round_2 > 0
round += 1
labels.push '2nd'
reviewer_data.push reviewer.avg_vol_in_round_2
all_reviewers_data.push @all_reviewers_avg_vol_in_round_2
end
if @all_reviewers_avg_vol_in_round_3 > 0
round += 1
labels.push '3rd'
reviewer_data.push reviewer.avg_vol_in_round_3
all_reviewers_data.push @all_reviewers_avg_vol_in_round_3
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

After:

def initialize_chart_elements(reviewer)
#round = 0
labels = []
reviewer_data = []
all_reviewers_data = []

(1..@assignment.num_review_rounds).each do |round_num|
#round += 1
labels.push round_num.to_s
reviewer_data.push eval("reviewer.avg_vol_in_round_#{round_num}")
all_reviewers_data.push instance_variable_get("@all_reviewers_avg_vol_in_round_#{round_num}")
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

3. Refractored review_mapping_helper.rb.
Refactored review_mapping_helper.rb. Few of the changes are reducing the length of method names, adding comments, removing redundant code,
adding string interpolation.

please see the pull request for detailed code. [https://github.com/expertiza/expertiza/pull/1493]

IndependentStudy Visualizations

2019-12-04T00:24:40Z

Sseelam2:

'''Introduction:'''
This project fixes the way graphs are represented in the review report. Expertiza interprets metrics in the form of a graph in the review report. This Project also refractors
review_mapping_helper.rb.

'''Problem Statement:'''
Currently, the bars in the graphs overlap and they are not positioned properly.
The legends (vol. and avg vol.) are present in every cell in the metrics column.
The current code assumes that there are no more than three rounds.
Removed charts_helpers.rb since it was not being used anywhere in the code.
Refractored review_mapping_helper.rb.
Add comments to methods.
'''
Files modified:'''
app/assets/stylesheets/legend.css
app/helpers/charts_helper.rb
app/helpers/review_mapping_helper.rb
app/views/reports/_calibration_report.html.erb
app/views/reports/_feedback_report.html.erb
app/views/reports/_review_report.html.erb
app/views/reports/_team_score.html.erb

'''Solution:'''
1. Currently, the bars in the graphs overlap and they are not positioned properly and the legends (vol. and avg vol.) are present in every cell in the metrics column.
This change is present in the files - app/views/reports/_review_report.html.erb and app/helpers/review_mapping_helper.rb

Before:
[[File:Before_Graph.jpg]]

After:
[[File:After_Graph.jpg]]

2. Current code assumes that there are no more than three rounds. Changes can be found in app/helpers/review_mapping_helper.rb

Before:
def initialize_chart_elements(reviewer)
round = 0
labels = []
reviewer_data = []
all_reviewers_data = []
if @all_reviewers_avg_vol_in_round_1 > 0
round += 1
labels.push '1st'
reviewer_data.push reviewer.avg_vol_in_round_1
all_reviewers_data.push @all_reviewers_avg_vol_in_round_1
end
if @all_reviewers_avg_vol_in_round_2 > 0
round += 1
labels.push '2nd'
reviewer_data.push reviewer.avg_vol_in_round_2
all_reviewers_data.push @all_reviewers_avg_vol_in_round_2
end
if @all_reviewers_avg_vol_in_round_3 > 0
round += 1
labels.push '3rd'
reviewer_data.push reviewer.avg_vol_in_round_3
all_reviewers_data.push @all_reviewers_avg_vol_in_round_3
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

After:

def initialize_chart_elements(reviewer)
#round = 0
labels = []
reviewer_data = []
all_reviewers_data = []

(1..@assignment.num_review_rounds).each do |round_num|
#round += 1
labels.push round_num.to_s
reviewer_data.push eval("reviewer.avg_vol_in_round_#{round_num}")
all_reviewers_data.push instance_variable_get("@all_reviewers_avg_vol_in_round_#{round_num}")
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

3. Refractored review_mapping_helper.rb.
Refactored review_mapping_helper.rb. Few of the changes are reducing the length of method names, adding comments, removing redundant code,
adding string interpolation.

please see the pull request for detailed code. [https://github.com/expertiza/expertiza/pull/1493]

File:After Graph.png

2019-12-04T00:23:46Z

Sseelam2:

IndependentStudy Visualizations

2019-12-04T00:21:13Z

Sseelam2:

'''Introduction:'''
This project fixes the way graphs are reperesented in the review report. Expertiza interprets metrics in the form of a graph in the review report. This Project also refractors
review_mapping_helper.rb.

'''Problem Statement:'''
Currently the bars in the graphs overlap and they are not positioned properly.
The legends (vol. and avg vol.) are present in every cell in the metrics column.
Current code assumes that there are no more than three rounds.
Removed charts_helpers.rb since it was not being used anywhere in the code.
Refractored review_mapping_helper.rb.
Add comments to methods.
'''
Files modified:'''
app/assets/stylesheets/legend.css
app/helpers/charts_helper.rb
app/helpers/review_mapping_helper.rb
app/views/reports/_calibration_report.html.erb
app/views/reports/_feedback_report.html.erb
app/views/reports/_review_report.html.erb
app/views/reports/_team_score.html.erb

'''Solution:'''
1. Currently the bars in the graphs overlap and they are not positioned properly and the legends (vol. and avg vol.) are present in every cell in the metrics column.
This change is present in the files - app/views/reports/_review_report.html.erb and app/helpers/review_mapping_helper.rb

Before:
[[File:Before_Graph.jpg]]
After:
[[File:after.jpg]]

2. Current code assumes that there are no more than three rounds. Changes can be found in app/helpers/review_mapping_helper.rb

Before:
def initialize_chart_elements(reviewer)
round = 0
labels = []
reviewer_data = []
all_reviewers_data = []
if @all_reviewers_avg_vol_in_round_1 > 0
round += 1
labels.push '1st'
reviewer_data.push reviewer.avg_vol_in_round_1
all_reviewers_data.push @all_reviewers_avg_vol_in_round_1
end
if @all_reviewers_avg_vol_in_round_2 > 0
round += 1
labels.push '2nd'
reviewer_data.push reviewer.avg_vol_in_round_2
all_reviewers_data.push @all_reviewers_avg_vol_in_round_2
end
if @all_reviewers_avg_vol_in_round_3 > 0
round += 1
labels.push '3rd'
reviewer_data.push reviewer.avg_vol_in_round_3
all_reviewers_data.push @all_reviewers_avg_vol_in_round_3
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

After:

def initialize_chart_elements(reviewer)
#round = 0
labels = []
reviewer_data = []
all_reviewers_data = []

(1..@assignment.num_review_rounds).each do |round_num|
#round += 1
labels.push round_num.to_s
reviewer_data.push eval("reviewer.avg_vol_in_round_#{round_num}")
all_reviewers_data.push instance_variable_get("@all_reviewers_avg_vol_in_round_#{round_num}")
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

3. Refractored review_mapping_helper.rb.
Refractored review_mapping_helper.rb. Few of the changes are reducing the length of method names, adding comments, removing redudant code,
adding string interpolation.

please see the pull request for detailed code. [https://github.com/expertiza/expertiza/pull/1493]

File:Before Graph.jpg

2019-12-04T00:20:21Z

Sseelam2: uploaded a new version of "File:Before Graph.jpg"

File:Before Graph.jpg

2019-12-04T00:17:16Z

Sseelam2: uploaded a new version of "File:Before Graph.jpg"

File:Before Graph.jpg

2019-12-04T00:12:07Z

Sseelam2: uploaded a new version of "File:Before Graph.jpg"

File:Before Graph.jpg

2019-12-04T00:06:36Z

Sseelam2:

IndependentStudy Visualizations

2019-12-04T00:04:45Z

Sseelam2:

'''Introduction:'''
This project fixes the way graphs are reperesented in the review report. Expertiza interprets metrics in the form of a graph in the review report. This Project also refractors
review_mapping_helper.rb.

'''Problem Statement:'''
Currently the bars in the graphs overlap and they are not positioned properly.
The legends (vol. and avg vol.) are present in every cell in the metrics column.
Current code assumes that there are no more than three rounds.
Removed charts_helpers.rb since it was not being used anywhere in the code.
Refractored review_mapping_helper.rb.
Add comments to methods.
'''
Files modified:'''
app/assets/stylesheets/legend.css
app/helpers/charts_helper.rb
app/helpers/review_mapping_helper.rb
app/views/reports/_calibration_report.html.erb
app/views/reports/_feedback_report.html.erb
app/views/reports/_review_report.html.erb
app/views/reports/_team_score.html.erb

'''Solution:'''
1. Currently the bars in the graphs overlap and they are not positioned properly and the legends (vol. and avg vol.) are present in every cell in the metrics column.
This change is present in the files - app/views/reports/_review_report.html.erb and app/helpers/review_mapping_helper.rb

Before:
[[File:Before.jpg]]
After:
[[File:after.jpg]]

2. Current code assumes that there are no more than three rounds. Changes can be found in app/helpers/review_mapping_helper.rb

Before:
def initialize_chart_elements(reviewer)
round = 0
labels = []
reviewer_data = []
all_reviewers_data = []
if @all_reviewers_avg_vol_in_round_1 > 0
round += 1
labels.push '1st'
reviewer_data.push reviewer.avg_vol_in_round_1
all_reviewers_data.push @all_reviewers_avg_vol_in_round_1
end
if @all_reviewers_avg_vol_in_round_2 > 0
round += 1
labels.push '2nd'
reviewer_data.push reviewer.avg_vol_in_round_2
all_reviewers_data.push @all_reviewers_avg_vol_in_round_2
end
if @all_reviewers_avg_vol_in_round_3 > 0
round += 1
labels.push '3rd'
reviewer_data.push reviewer.avg_vol_in_round_3
all_reviewers_data.push @all_reviewers_avg_vol_in_round_3
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

After:

def initialize_chart_elements(reviewer)
#round = 0
labels = []
reviewer_data = []
all_reviewers_data = []

(1..@assignment.num_review_rounds).each do |round_num|
#round += 1
labels.push round_num.to_s
reviewer_data.push eval("reviewer.avg_vol_in_round_#{round_num}")
all_reviewers_data.push instance_variable_get("@all_reviewers_avg_vol_in_round_#{round_num}")
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

3. Refractored review_mapping_helper.rb.
Refractored review_mapping_helper.rb. Few of the changes are reducing the length of method names, adding comments, removing redudant code,
adding string interpolation.

please see the pull request for detailed code. [https://github.com/expertiza/expertiza/pull/1493]

IndependentStudy Visualizations

2019-12-03T23:58:36Z

Sseelam2: Created page with "Introduction This project fixes the way graphs are reperesented in the review report. Expertiza interprets metrics in the form of a graph in the review report.This Project also r..."

Introduction
This project fixes the way graphs are reperesented in the review report. Expertiza interprets metrics in the form of a graph in the review report.This Project also refractors
review_mapping_helper.rb.

Problem Statement
Currently the bars in the graphs overlap and they are not positioned properly.
The legends (vol. and avg vol.) are present in every cell in the metrics column.
Current code assumes that there are no more than three rounds.
Removed charts_helpers.rb since it was not being used anywhere in the code.
Refractored review_mapping_helper.rb.
Add comments to methods.

Files modified:
app/assets/stylesheets/legend.css
app/helpers/charts_helper.rb
app/helpers/review_mapping_helper.rb
app/views/reports/_calibration_report.html.erb
app/views/reports/_feedback_report.html.erb
app/views/reports/_review_report.html.erb
app/views/reports/_team_score.html.erb

Solution
1.Currently the bars in the graphs overlap and they are not positioned properly and the legends (vol. and avg vol.) are present in every cell in the metrics column.
This change is present in the files - app/views/reports/_review_report.html.erb and app/helpers/review_mapping_helper.rb

Before:
pic
After:
pic

2.Current code assumes that there are no more than three rounds. Changes can be found in app/helpers/review_mapping_helper.rb

before:
def initialize_chart_elements(reviewer)
round = 0
labels = []
reviewer_data = []
all_reviewers_data = []
if @all_reviewers_avg_vol_in_round_1 > 0
round += 1
labels.push '1st'
reviewer_data.push reviewer.avg_vol_in_round_1
all_reviewers_data.push @all_reviewers_avg_vol_in_round_1
end
if @all_reviewers_avg_vol_in_round_2 > 0
round += 1
labels.push '2nd'
reviewer_data.push reviewer.avg_vol_in_round_2
all_reviewers_data.push @all_reviewers_avg_vol_in_round_2
end
if @all_reviewers_avg_vol_in_round_3 > 0
round += 1
labels.push '3rd'
reviewer_data.push reviewer.avg_vol_in_round_3
all_reviewers_data.push @all_reviewers_avg_vol_in_round_3
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

After:

def initialize_chart_elements(reviewer)
#round = 0
labels = []
reviewer_data = []
all_reviewers_data = []

(1..@assignment.num_review_rounds).each do |round_num|
#round += 1
labels.push round_num.to_s
reviewer_data.push eval("reviewer.avg_vol_in_round_#{round_num}")
all_reviewers_data.push instance_variable_get("@all_reviewers_avg_vol_in_round_#{round_num}")
end

labels.push 'Total'
reviewer_data.push reviewer.overall_avg_vol
all_reviewers_data.push @all_reviewers_overall_avg_vol
[labels, reviewer_data, all_reviewers_data]
end

3. Refractored review_mapping_helper.rb.
Refractored review_mapping_helper.rb. Few of the changes are reducing the length of method names, adding comments, removing redudant code,
adding string interpolation.

please see the pull request for detailed code. https://github.com/expertiza/expertiza/pull/1493

CSC/ECE 517 Fall 2018 - E1825: Add past-due assignments to task list

2018-10-28T15:37:05Z

Sseelam2: Created page with "About Expertiza Expertiza is a web application where students can submit and peer-review learning objects (articles, code, web sites, etc). It is used in select courses at NC Sta..."

About Expertiza
Expertiza is a web application where students can submit and peer-review learning objects (articles, code, web sites, etc). It is used in select courses at NC State and by professors at several other colleges and universities. -- Expertiza project