Merge pull request #353 from edx/dsjen/avg-video-calc

Fixed video charts to use average incomplete views.

analytics_dashboard/courses/presenters/engagement.py

@@ -146,7 +146,7 @@ class CourseEngagementVideoPresenter(CourseAPIPresenterMixin, BasePresenter):
             'users_at_start': total_start_users,
             'users_at_end': total_end_users,
             'index': index + 1,
-            'average_users_at_start': 0,
+            'average_start_only_users': 0,
             'average_users_at_end': 0,
         })
 
@@ -158,7 +158,7 @@ class CourseEngagementVideoPresenter(CourseAPIPresenterMixin, BasePresenter):
         if has_views and parent['num_modules']:
             num_modules = float(parent['num_modules'])
             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,
             })
 

analytics_dashboard/courses/tests/test_presenters.py

@@ -233,7 +233,7 @@ class CourseEngagementVideoPresenterTests(SwitchMixin, TestCase):
             'users_at_start': 60,
             'users_at_end': 40,
             'index': 1,
-            'average_users_at_start': 30,
+            'average_start_only_users': 10,
             'average_users_at_end': 20,
             'end_percent': 2/3,
             'start_only_users': 20,

analytics_dashboard/static/js/engagement-video-content-main.js

@@ -9,7 +9,7 @@ require(['vendor/domReady!', 'load/init-page'], function (doc, page) {
             var model = page.models.courseModel,
                 graphVideoColumns = [
                     {
-                        key: 'average_users_at_start',
+                        key: 'average_users_at_end',
                         percent_key: 'end_percent',
                         title: gettext('Average Complete Views'),
                         className: 'text-right',
@@ -18,7 +18,7 @@ require(['vendor/domReady!', 'load/init-page'], function (doc, page) {
                         color: '#58BC4B'
                     },
                     {
-                        key: 'average_users_at_end',
+                        key: 'average_start_only_users',
                         percent_key: 'start_only_percent',
                         title: gettext('Average Incomplete Views'),
                         className: 'text-right',