Skip to content

Commit 66e493a

Browse files
committed
refactor: use course_run instead of run_name
This change upgrades to the latest version of aspects-dbt and uses the already-parsed `course_run` field instead of the ad-hoc `run_name` field. This change also includes updates to the filter name and to the queries that rely on that filter.
1 parent 8a5c236 commit 66e493a

19 files changed

+54
-78
lines changed

tutoraspects/plugin.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@
214214
# For now we are pulling this from github, which should allow maximum
215215
# flexibility for forking, running branches, specific versions, etc.
216216
("DBT_REPOSITORY", "https://github.com/openedx/aspects-dbt"),
217-
("DBT_BRANCH", "v1.3.1"),
217+
("DBT_BRANCH", "v2.0.0"),
218218
# Path to the dbt project inside the repository
219219
("DBT_REPOSITORY_PATH", "aspects-dbt/aspects"),
220220
# This is a pip compliant list of Python packages to install to run dbt

tutoraspects/templates/openedx-assets/assets/dashboards/Instructor_dashboard.yaml

+6-6
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ metadata:
7474
description: ''
7575
filterType: filter_select
7676
id: NATIVE_FILTER-Vx7HxG8_7
77-
name: org
77+
name: Org
7878
requiredFirst: true
7979
scope:
8080
excluded: []
@@ -89,7 +89,7 @@ metadata:
8989
targets:
9090
- column:
9191
name: org
92-
datasetUuid: 4b274428-d781-41be-b362-ed6917443678
92+
datasetUuid: 41278a97-d0ff-4645-9514-d79f80d275df
9393
type: NATIVE_FILTER
9494
- cascadeParentIds:
9595
- NATIVE_FILTER-Vx7HxG8_7
@@ -131,7 +131,7 @@ metadata:
131131
targets:
132132
- column:
133133
name: course_name
134-
datasetUuid: 4b274428-d781-41be-b362-ed6917443678
134+
datasetUuid: 41278a97-d0ff-4645-9514-d79f80d275df
135135
type: NATIVE_FILTER
136136
- cascadeParentIds:
137137
- NATIVE_FILTER-Vx7HxG8_7
@@ -160,7 +160,7 @@ metadata:
160160
description: ''
161161
filterType: filter_select
162162
id: NATIVE_FILTER-E6-vOpjZv
163-
name: Run name
163+
name: Course run
164164
scope:
165165
excluded: []
166166
rootPath:
@@ -173,8 +173,8 @@ metadata:
173173
- TAB-CLiLC4zxo
174174
targets:
175175
- column:
176-
name: run_name
177-
datasetUuid: 4b274428-d781-41be-b362-ed6917443678
176+
name: course_run
177+
datasetUuid: 41278a97-d0ff-4645-9514-d79f80d275df
178178
type: NATIVE_FILTER
179179
- cascadeParentIds:
180180
- NATIVE_FILTER-Vx7HxG8_7

tutoraspects/templates/openedx-assets/assets/datasets/dim_courses.yaml tutoraspects/templates/openedx-assets/assets/datasets/course_names.yaml

+11-11
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
_file_name: dim_courses.yaml
1+
_file_name: course_names.yaml
22
cache_timeout: null
33
columns:
44
- advanced_data_type: null
@@ -14,7 +14,7 @@ columns:
1414
type: String
1515
verbose_name: null
1616
- advanced_data_type: null
17-
column_name: run_name
17+
column_name: course_key
1818
description: null
1919
expression: null
2020
extra: null
@@ -26,7 +26,7 @@ columns:
2626
type: String
2727
verbose_name: null
2828
- advanced_data_type: null
29-
column_name: course_key
29+
column_name: course_run
3030
description: null
3131
expression: null
3232
extra: null
@@ -59,17 +59,17 @@ main_dttm_col: null
5959
metrics:
6060
- d3format: null
6161
description: null
62-
expression: count(*)
62+
expression: COUNT(*)
6363
extra: null
6464
metric_name: count
65-
metric_type: null
66-
verbose_name: null
65+
metric_type: count
66+
verbose_name: COUNT(*)
6767
warning_text: null
6868
offset: 0
6969
params: null
70-
schema: null
71-
sql: "{% include 'openedx-assets/queries/dim_courses.sql' %}"
72-
table_name: dim_courses
73-
template_params: {}
74-
uuid: 4b274428-d781-41be-b362-ed6917443678
70+
schema: {{ ASPECTS_EVENT_SINK_DATABASE }}
71+
sql: null
72+
table_name: course_names
73+
template_params: null
74+
uuid: 41278a97-d0ff-4645-9514-d79f80d275df
7575
version: 1.0.0

tutoraspects/templates/openedx-assets/assets/datasets/dim_course_problems.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ columns:
2626
type: String
2727
verbose_name: null
2828
- advanced_data_type: null
29-
column_name: run_name
29+
column_name: course_run
3030
description: null
3131
expression: null
3232
extra: null

tutoraspects/templates/openedx-assets/assets/datasets/dim_course_videos.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ columns:
2626
type: String
2727
verbose_name: null
2828
- advanced_data_type: null
29-
column_name: run_name
29+
column_name: course_run
3030
description: null
3131
expression: null
3232
extra: null

tutoraspects/templates/openedx-assets/assets/datasets/fact_enrollments.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ columns:
6262
type: String
6363
verbose_name: null
6464
- advanced_data_type: null
65-
column_name: run_name
65+
column_name: course_run
6666
description: null
6767
expression: null
6868
extra: null

tutoraspects/templates/openedx-assets/assets/datasets/fact_enrollments_by_day.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ columns:
6262
type: String
6363
verbose_name: null
6464
- advanced_data_type: null
65-
column_name: run_name
65+
column_name: course_run
6666
description: null
6767
expression: null
6868
extra: null

tutoraspects/templates/openedx-assets/assets/datasets/fact_learner_problem_summary.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ columns:
8686
type: String
8787
verbose_name: null
8888
- advanced_data_type: null
89-
column_name: run_name
89+
column_name: course_run
9090
description: null
9191
expression: null
9292
extra: null

tutoraspects/templates/openedx-assets/assets/datasets/fact_problem_responses.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ columns:
7474
type: String
7575
verbose_name: null
7676
- advanced_data_type: null
77-
column_name: run_name
77+
column_name: course_run
7878
description: null
7979
expression: null
8080
extra: null

tutoraspects/templates/openedx-assets/assets/datasets/fact_transcript_usage.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ columns:
5050
type: String
5151
verbose_name: null
5252
- advanced_data_type: null
53-
column_name: run_name
53+
column_name: course_run
5454
description: null
5555
expression: null
5656
extra: {}

tutoraspects/templates/openedx-assets/assets/datasets/fact_video_plays.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ columns:
5050
type: String
5151
verbose_name: null
5252
- advanced_data_type: null
53-
column_name: run_name
53+
column_name: course_run
5454
description: null
5555
expression: null
5656
extra: {}

tutoraspects/templates/openedx-assets/assets/datasets/fact_watched_video_segments.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ columns:
6262
type: String
6363
verbose_name: null
6464
- advanced_data_type: null
65-
column_name: run_name
65+
column_name: course_run
6666
description: null
6767
expression: null
6868
extra: {}

tutoraspects/templates/openedx-assets/assets/datasets/hints_per_success.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ columns:
5050
type: String
5151
verbose_name: null
5252
- advanced_data_type: null
53-
column_name: run_name
53+
column_name: course_run
5454
description: null
5555
expression: null
5656
extra: null

tutoraspects/templates/openedx-assets/queries/dim_course_problems.sql

+8-10
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
1-
with courses as (
2-
{% include 'openedx-assets/queries/dim_courses.sql' %}
3-
), problems as (
1+
with problems as (
42
select
53
org,
64
course_key,
@@ -21,14 +19,14 @@ with courses as (
2119
)
2220

2321
select
24-
courses.org as org,
25-
courses.course_name as course_name,
26-
courses.course_key as course_key,
27-
courses.run_name as run_name,
22+
course_names.org as org,
23+
course_names.course_name as course_name,
24+
course_names.course_key as course_key,
25+
course_names.course_run as course_run,
2826
problems.problem_id as problem_id,
2927
problems.problem_name as problem_name
3028
from
3129
problems
32-
join courses
33-
on (problems.org = courses.org
34-
and problems.course_key = courses.course_key)
30+
join {{ ASPECTS_EVENT_SINK_DATABASE }}.course_names course_names
31+
on (problems.org = course_names.org
32+
and problems.course_key = course_names.course_key)

tutoraspects/templates/openedx-assets/queries/dim_course_videos.sql

+8-10
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
1-
with courses as (
2-
{% include 'openedx-assets/queries/dim_courses.sql' %}
3-
), video_blocks as (
1+
with video_blocks as (
42
select
53
org,
64
course_key,
@@ -21,14 +19,14 @@ with courses as (
2119
)
2220

2321
select
24-
courses.org as org,
25-
courses.course_name as course_name,
26-
courses.course_key as course_key,
27-
courses.run_name as run_name,
22+
course_names.org as org,
23+
course_names.course_name as course_name,
24+
course_names.course_key as course_key,
25+
course_names.course_run as course_run,
2826
video_blocks.video_id as video_id,
2927
video_blocks.video_name as video_name
3028
from
31-
courses
29+
{{ ASPECTS_EVENT_SINK_DATABASE }}.course_names course_names
3230
join video_blocks
33-
on (courses.org = video_blocks.org
34-
and courses.course_key = video_blocks.course_key)
31+
on (course_names.org = video_blocks.org
32+
and course_names.course_key = video_blocks.course_key)

tutoraspects/templates/openedx-assets/queries/dim_courses.sql

-20
This file was deleted.

tutoraspects/templates/openedx-assets/queries/fact_enrollments_by_day.sql

+6-6
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,11 @@ with enrollments_ranked as (
33
emission_time,
44
org,
55
course_name,
6-
run_name,
6+
course_run,
77
actor_id,
88
enrollment_mode,
99
enrollment_status,
10-
rank() over (partition by date(emission_time), org, course_name, run_name, actor_id order by emission_time desc) as event_rank
10+
rank() over (partition by date(emission_time), org, course_name, course_run, actor_id order by emission_time desc) as event_rank
1111
from
1212
{{ DBT_PROFILE_TARGET_DATABASE }}.fact_enrollments
1313
{% raw -%}
@@ -20,12 +20,12 @@ with enrollments_ranked as (
2020
select
2121
org,
2222
course_name,
23-
run_name,
23+
course_run,
2424
actor_id,
2525
enrollment_status,
2626
enrollment_mode,
2727
emission_time as window_start_at,
28-
lagInFrame(emission_time, 1, now() + interval '1' day) over (partition by org, course_name, run_name, actor_id order by emission_time desc) as window_end_at
28+
lagInFrame(emission_time, 1, now() + interval '1' day) over (partition by org, course_name, course_run, actor_id order by emission_time desc) as window_end_at
2929
from
3030
enrollments_ranked
3131
where
@@ -34,7 +34,7 @@ with enrollments_ranked as (
3434
select
3535
org,
3636
course_name,
37-
run_name,
37+
course_run,
3838
actor_id,
3939
enrollment_status,
4040
enrollment_mode,
@@ -54,7 +54,7 @@ select
5454
)) as enrollment_status_date,
5555
org,
5656
course_name,
57-
run_name,
57+
course_run,
5858
actor_id,
5959
enrollment_status,
6060
enrollment_mode

tutoraspects/templates/openedx-assets/queries/fact_watched_video_segments.sql

+2-2
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ with starts as (
6060
select
6161
segments.org as org,
6262
courses.course_name as course_name,
63-
splitByString('+', courses.course_key)[-1] as run_name,
63+
courses.course_run as course_run,
6464
blocks.block_name as video_name,
6565
segments.actor_id as actor_id,
6666
segments.started_at as started_at,
@@ -77,7 +77,7 @@ with starts as (
7777
select
7878
org,
7979
course_name,
80-
run_name,
80+
course_run,
8181
video_name,
8282
actor_id,
8383
started_at,
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
select
22
org,
33
course_name,
4-
run_name,
4+
course_run,
55
problem_name,
66
actor_id,
77
sum(num_hints_displayed) + sum(num_answers_displayed) as total_hints
@@ -11,6 +11,6 @@ where success = 1
1111
group by
1212
org,
1313
course_name,
14-
run_name,
14+
course_run,
1515
problem_name,
1616
actor_id

0 commit comments

Comments
 (0)