Commit 73e13497 by Dennis Jen

Fixed video charts to use average incomplete views.

parent 9f1c002b
...@@ -146,7 +146,7 @@ class CourseEngagementVideoPresenter(CourseAPIPresenterMixin, BasePresenter): ...@@ -146,7 +146,7 @@ class CourseEngagementVideoPresenter(CourseAPIPresenterMixin, BasePresenter):
'users_at_start': total_start_users, 'users_at_start': total_start_users,
'users_at_end': total_end_users, 'users_at_end': total_end_users,
'index': index + 1, 'index': index + 1,
'average_users_at_start': 0, 'average_start_only_users': 0,
'average_users_at_end': 0, 'average_users_at_end': 0,
}) })
...@@ -158,7 +158,7 @@ class CourseEngagementVideoPresenter(CourseAPIPresenterMixin, BasePresenter): ...@@ -158,7 +158,7 @@ class CourseEngagementVideoPresenter(CourseAPIPresenterMixin, BasePresenter):
if has_views and parent['num_modules']: if has_views and parent['num_modules']:
num_modules = float(parent['num_modules']) num_modules = float(parent['num_modules'])
parent.update({ parent.update({
'average_users_at_start': total_start_users / num_modules, 'average_start_only_users': parent['start_only_users'] / num_modules,
'average_users_at_end': total_end_users / num_modules, 'average_users_at_end': total_end_users / num_modules,
}) })
......
...@@ -233,7 +233,7 @@ class CourseEngagementVideoPresenterTests(SwitchMixin, TestCase): ...@@ -233,7 +233,7 @@ class CourseEngagementVideoPresenterTests(SwitchMixin, TestCase):
'users_at_start': 60, 'users_at_start': 60,
'users_at_end': 40, 'users_at_end': 40,
'index': 1, 'index': 1,
'average_users_at_start': 30, 'average_start_only_users': 10,
'average_users_at_end': 20, 'average_users_at_end': 20,
'end_percent': 2/3, 'end_percent': 2/3,
'start_only_users': 20, 'start_only_users': 20,
......
...@@ -9,7 +9,7 @@ require(['vendor/domReady!', 'load/init-page'], function (doc, page) { ...@@ -9,7 +9,7 @@ require(['vendor/domReady!', 'load/init-page'], function (doc, page) {
var model = page.models.courseModel, var model = page.models.courseModel,
graphVideoColumns = [ graphVideoColumns = [
{ {
key: 'average_users_at_start', key: 'average_users_at_end',
percent_key: 'end_percent', percent_key: 'end_percent',
title: gettext('Average Complete Views'), title: gettext('Average Complete Views'),
className: 'text-right', className: 'text-right',
...@@ -18,7 +18,7 @@ require(['vendor/domReady!', 'load/init-page'], function (doc, page) { ...@@ -18,7 +18,7 @@ require(['vendor/domReady!', 'load/init-page'], function (doc, page) {
color: '#58BC4B' color: '#58BC4B'
}, },
{ {
key: 'average_users_at_end', key: 'average_start_only_users',
percent_key: 'start_only_percent', percent_key: 'start_only_percent',
title: gettext('Average Incomplete Views'), title: gettext('Average Incomplete Views'),
className: 'text-right', className: 'text-right',
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment