fix: count distince videos

openedx · Nov 21, 2024 · 5b6bb66 · 5b6bb66
1 parent d735fc4
commit 5b6bb66
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 16 deletions.
diff --git a/.../aspects/build/aspects-superset/openedx-assets/assets/charts/Video_Engagement_5b00f7.yaml b/.../aspects/build/aspects-superset/openedx-assets/assets/charts/Video_Engagement_5b00f7.yaml
@@ -7,6 +7,7 @@ description: null
 params:
   adhoc_filters: []
   all_columns: []
+  annotation_layers: []
   color_pn: true
   column_config:
     rewatched_percent:
@@ -41,15 +42,14 @@ query_context:
   form_data:
     adhoc_filters: []
     all_columns: []
+    annotation_layers: []
     color_pn: true
     column_config:
       rewatched_percent:
         d3NumberFormat: ',.1%'
       watched_percent:
         d3NumberFormat: ',.1%'
     conditional_formatting: []
-    dashboards:
-    - 241
     datasource: 161__table
     extra_form_data: {}
     force: false

diff --git a/.../aspects/build/aspects-superset/openedx-assets/assets/charts/Video_Engagement_6b7610.yaml b/.../aspects/build/aspects-superset/openedx-assets/assets/charts/Video_Engagement_6b7610.yaml
@@ -7,6 +7,7 @@ description: null
 params:
   adhoc_filters: []
   all_columns: []
+  annotation_layers: []
   color_pn: true
   column_config:
     rewatched_percent:

diff --git a/...aspects/build/aspects-superset/openedx-assets/assets/datasets/watched_video_duration.yaml b/...aspects/build/aspects-superset/openedx-assets/assets/datasets/watched_video_duration.yaml
@@ -12,7 +12,7 @@ columns:
   is_active: true
   is_dttm: false
   python_date_format: null
-  type: Decimal(38, 2)
+  type: Decimal(9, 2)
   verbose_name: Rewatched Time
 - advanced_data_type: null
   column_name: watched_time
@@ -24,7 +24,7 @@ columns:
   is_active: true
   is_dttm: false
   python_date_format: null
-  type: Decimal(38, 2)
+  type: Decimal(9, 2)
   verbose_name: Watched Time
 - advanced_data_type: null
   column_name: course_name
@@ -111,7 +111,7 @@ columns:
   type: String
   verbose_name: Org
 - advanced_data_type: null
-  column_name: video_count_watched
+  column_name: object_id
   description: null
   expression: null
   extra: {}
@@ -120,8 +120,8 @@ columns:
   is_active: true
   is_dttm: false
   python_date_format: null
-  type: UInt64
-  verbose_name: Video Count Watched
+  type: String
+  verbose_name: Object Id
 database_uuid: 21174b6c-4d40-4958-8161-d6c3cf5e77b6
 default_endpoint: null
 description: null
@@ -140,6 +140,16 @@ metrics:
   metric_type: null
   verbose_name: '% Video Seconds Rewatched'
   warning_text: null
+- currency: null
+  d3format: null
+  description: null
+  expression: count(distinct object_id)
+  extra:
+    warning_markdown: ''
+  metric_name: num_videos_watched
+  metric_type: null
+  verbose_name: Number of Videos Watched
+  warning_text: null
 - currency: null
   d3format: null
   description: null
@@ -160,15 +170,6 @@ metrics:
   metric_type: null
   verbose_name: Avg Video Length (seconds)
   warning_text: null
-- currency: null
-  d3format: null
-  description: null
-  expression: sum(video_count_watched)
-  extra: {}
-  metric_name: num_videos_watched
-  metric_type: null
-  verbose_name: Number of Videos Watched
-  warning_text: null
 - currency: null
   d3format: null
   description: null