Show graphs for the lesson video statistics

5 files changed, 99 insertions, 21 deletions

diff --git a/stats/models.py b/stats/models.py
index 56c7f0d..60bc7bd 100644
--- a/stats/models.py
+++ b/stats/models.py
@@ -24,6 +24,22 @@ def time_to_seconds(time):
                               seconds=time.second).total_seconds()
 
 
+class TrackLessonManager(models.Manager):
+    def get_percentage_data(self, tracked_lessons):
+        percentage_data = {"1": 0, "2": 0, "3": 0, "4": 0}
+        for tracked in tracked_lessons:
+            percent = tracked.get_percentage_complete()
+            if percent < 25:
+                percentage_data["1"] = percentage_data["1"] + 1
+            elif percent >= 25 and percent < 50:
+                percentage_data["2"] = percentage_data["2"] + 1
+            elif percent >= 50 and percent < 75:
+                percentage_data["3"] = percentage_data["3"] + 1
+            elif percent >= 75:
+                percentage_data["4"] = percentage_data["4"] + 1
+        return percentage_data
+
+
 class TrackLesson(models.Model):
     user = models.ForeignKey(User, on_delete=models.CASCADE)
     course = models.ForeignKey(Course, on_delete=models.CASCADE)
@@ -33,6 +49,8 @@ class TrackLesson(models.Model):
     creation_time = models.DateTimeField(auto_now_add=True)
     watched = models.BooleanField(default=False)
 
+    objects = TrackLessonManager()
+
     class Meta:
         unique_together = ('user', 'course', 'lesson')
 
@@ -41,12 +59,12 @@ class TrackLesson(models.Model):
 
     def get_current_time(self):
         if self.current_time == '00:00:00':
-            return 'just started'
+            return '00:00:00'
         return self.current_time
 
     def get_video_duration(self):
         if self.video_duration == '00:00:00':
-            return 'will be available after 25% completion'
+            return '00:00:00'
         return self.video_duration
 
     def set_current_time(self, ct):
@@ -58,13 +76,13 @@ class TrackLesson(models.Model):
 
     def get_percentage_complete(self):
         if self.current_time == '00:00:00' and self.video_duration == '00:00:00':
-            return 'less than 25%'
+            return 0
         duration = str_to_time(self.video_duration)
         watch_time = str_to_time(self.current_time)
         duration_seconds = time_to_seconds(duration)
         watched_seconds = time_to_seconds(watch_time)
         percentage = round((watched_seconds / duration_seconds) * 100)
-        return 'approx {0} %'.format(percentage)
+        return percentage
 
 
     def get_last_access_time(self):
@@ -78,7 +96,7 @@ class TrackLesson(models.Model):
         if self.current_time != '00:00:00' and self.video_duration != '00:00:00':
             duration = str_to_time(self.video_duration)
             watch_time = (str_to_datetime(self.current_time) + timezone.timedelta(
-                seconds=10)).time()
+                seconds=120)).time()
             self.watched = watch_time >= duration
 
     def get_watched(self):
diff --git a/stats/templates/view_lesson_tracking.html b/stats/templates/view_lesson_tracking.html
index d8d35c2..a9d3025 100644
--- a/stats/templates/view_lesson_tracking.html
+++ b/stats/templates/view_lesson_tracking.html
@@ -1,6 +1,6 @@
 {% extends "manage.html" %}
 {% load static %}
-{% block title %} Lesson Views {% endblock %}
+{% block title %} Lesson Video Stats {% endblock %}
 {% block script %}
 <script type="text/javascript" src="{% static 'yaksh/js/jquery.tablesorter.min.js' %}">
 </script>
@@ -33,7 +33,58 @@
     <br><br>
     {% include "yaksh/paginator.html" %}
     <br>
-    <h4><strong>{{total}} student(s) viewed this lesson</strong></h4>
+        <div class="row">
+        <div class="col" id='barDiv1'></div>
+        <div class="col" id="barDiv2"></div>
+        <div class="col" id="barDiv3"></div>
+        </div>
+        <script type="text/javascript">
+            var config = {responsive: true}
+            var data = [
+                {
+                    x: ["Completed", "Not Completed"],
+                    y: ["{{completion.0}}", "{{completion.1}}"],
+                    type: 'bar'
+                }
+            ];
+            var layout = {
+                title: "Number of completions (Out of {{visits.2}})",
+                xaxis: {title: 'Completion status'},
+                yaxis: {title: 'Count'},
+                width: 400,
+                height: 400,
+            };
+            Plotly.newPlot('barDiv1', data, layout, config);
+            var data = [
+                {
+                    x: ["Visited", "Not Visited"],
+                    y: ["{{visits.0}}", "{{visits.1}}"],
+                    type: 'bar'
+                }
+            ];
+            var layout = {
+                title: "Number of visits (Out of {{visits.2}})",
+                xaxis: {title: 'Visit status'},
+                yaxis: {title: 'Count'},
+                width: 400,
+                height: 400,
+            };
+            Plotly.newPlot('barDiv2', data, layout, config);
+            var x_data = ["0-25", "25-50", "50-75", "75-100"], y_data = [];
+            {% for i, j in percentage_data.items %}
+                y_data.push("{{j}}")
+            {% endfor %}
+            var data = [{x: x_data, y: y_data, type: 'bar'}];
+            var layout = {
+                title: "Range wise completion (Out of {{total}})",
+                xaxis: {title: 'Percentage Range'},
+                yaxis: {title: 'Count'},
+                width: 400,
+                height: 400,
+            };
+            Plotly.newPlot('barDiv3', data, layout, config);
+        </script>
+    <br>
     <table class="table table-responsive" id="stats-table">
         <thead>
             <tr>
@@ -57,13 +108,13 @@
                 <td>{{track.creation_time}}</td>
                 <td>{{track.get_current_time}}</td>
                 <td>{{track.get_video_duration}}</td>
-                <td>{{track.get_percentage_complete}}</td>
+                <td>{{track.get_percentage_complete}} %</td>
                 <td>
                     {% with track.get_watched as watched %}
                         {% if watched %}
                             <span class="badge-pill badge-success">{{watched}}</span>
                         {% else %}
-                            <span class="badge-pill badge-success">{{watched}}</span>
+                            <span class="badge-pill badge-warning">{{watched}}</span>
                         {% endif %}
                     {% endwith %}
                 </td>
diff --git a/stats/views.py b/stats/views.py
index 53b7cf7..a5cdeb7 100644
--- a/stats/views.py
+++ b/stats/views.py
@@ -27,14 +27,14 @@ def add_tracker(request, tracker_id):
     if current_time:
         track.set_current_time(current_time)
         track.video_duration = video_duration
-        LessonLog.objects.create(
-            track_id=track.id, current_time=current_time,
-            last_access_time=timezone.now()
-        )
         track.save()
         if not track.watched:
             track.set_watched()
             track.save()
+        LessonLog.objects.create(
+            track_id=track.id, current_time=current_time,
+            last_access_time=timezone.now()
+        )
         success = True
     else:
         success = False
@@ -46,16 +46,25 @@ def add_tracker(request, tracker_id):
 @email_verified
 def view_lesson_watch_stats(request, course_id, lesson_id):
     user = request.user
-    course = get_object_or_404(Course, pk=course_id)
+    course = get_object_or_404(
+        Course.objects.prefetch_related("students"), id=course_id
+    )
     if not course.is_creator(user) and not course.is_teacher(user):
         raise Http404('This course does not belong to you')
     trackings = TrackLesson.objects.get_queryset().filter(
         course_id=course_id, lesson_id=lesson_id
     ).order_by("id")
-    total = trackings.count()
+    percentage_data = TrackLesson.objects.get_percentage_data(trackings)
+    visited = trackings.count()
+    completed = trackings.filter(watched=True).count()
+    students_total = course.students.count()
     paginator = Paginator(trackings, 30)
     page = request.GET.get('page')
     trackings = paginator.get_page(page)
-    context = {'objects': trackings, 'total': total, 'course_id': course_id,
-               'lesson_id': lesson_id}
+    context = {
+        'objects': trackings, 'total': visited, 'course_id': course_id,
+        'lesson_id': lesson_id, "percentage_data": percentage_data,
+        'completion': [completed, students_total-completed, students_total],
+        'visits': [visited, students_total-visited, students_total]
+    }
     return render(request, 'view_lesson_tracking.html', context)
diff --git a/yaksh/templates/yaksh/show_lesson_statistics.html b/yaksh/templates/yaksh/show_lesson_statistics.html
index a7c2ebd..dfce52c 100644
--- a/yaksh/templates/yaksh/show_lesson_statistics.html
+++ b/yaksh/templates/yaksh/show_lesson_statistics.html
@@ -1,7 +1,7 @@
 {% extends "manage.html" %}
 {% load static %}
 {% load custom_filters %}
-{% block title %} Lesson Statistics {% endblock %}
+{% block title %} Lesson Quiz Stats {% endblock %}
 {% block pagetitle %} Statistics for {{lesson}} {% endblock %}
 {% block script %}
 <script type="text/javascript" src="{% static 'yaksh/js/jquery.tablesorter.min.js' %}">
@@ -135,8 +135,8 @@
                                 <div class="progress" style="width: 30%">
                                 {% if percent %}
                                     <div class="progress-bar bg-success" role="progressbar" aria-valuenow="{{percent}}"
-                                      aria-valuemin="0" aria-valuemax="100" style="width:{{percent}}%">
-                                        <b style="color: white;">{{percent}}%</b>
+                                      aria-valuemin="0" aria-valuemax="100" style="width:{{percent|floatformat}}%">
+                                        <b style="color: white;">{{percent|floatformat}}%</b>
                                     </div>
                                 {% else %}
                                     <b style="color: black;">0%</b>
diff --git a/yaksh/templatetags/custom_filters.py b/yaksh/templatetags/custom_filters.py
index 91681c2..7eba939 100644
--- a/yaksh/templatetags/custom_filters.py
+++ b/yaksh/templatetags/custom_filters.py
@@ -209,5 +209,5 @@ def get_tc_percent(tc_id, data):
 def get_lesson_views(course_id, lesson_id):
     course = Course.objects.get(id=course_id)
     return TrackLesson.objects.filter(
-        course_id=course_id, lesson_id=lesson_id
+        course_id=course_id, lesson_id=lesson_id, watched=True
     ).count(), course.students.count()