Skip to content

Commit

Permalink
fix: Operator dash user counts, performance
Browse files Browse the repository at this point in the history
Don't use the user_pii table for user counts, since it may not be populated. Change this chart to be a count of active enrollments, which people have said is more useful.

Also changed the time range for operator dash to 90 days to improve performance of the actors / events graphs.
  • Loading branch information
bmtcril committed May 29, 2024
1 parent 04454b1 commit 17a05e8
Show file tree
Hide file tree
Showing 13 changed files with 241 additions and 224 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ _file_name: Active_Courses.yaml
cache_timeout: null
certification_details: null
certified_by: null
dataset_uuid: f13a4d42-2f44-58a4-99d2-2f480d63fdf1
dataset_uuid: f89c07c4-aa3f-527e-84fb-3d4e71ffc777
description: null
params:
adhoc_filters:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ _file_name: Active_Learners.yaml
cache_timeout: null
certification_details: null
certified_by: null
dataset_uuid: 5d7fa3b0-5692-5229-b8fc-4711b88d5401
dataset_uuid: 458b2b37-d403-5063-afd9-7b1c5cac35a5
description: null
params:
adhoc_filters:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ _file_name: Course_Enrollments_Over_Time.yaml
cache_timeout: null
certification_details: null
certified_by: null
dataset_uuid: c592f3b1-6a99-5aea-8342-a446c4e36a67
dataset_uuid: a9fb7758-9da0-5b57-9787-f6c0ac3d6ec6
description: null
params:
adhoc_filters:
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,134 @@
_file_name: Currently_Registered_Users.yaml
cache_timeout: null
certification_details: null
certified_by: null
dataset_uuid: 3dde421d-c0d0-43ab-8087-ca470985e88e
description: null
params:
adhoc_filters:
- clause: WHERE
comparator: No filter
datasourceWarning: false
expressionType: SIMPLE
filterOptionName: filter_twlv610ia5j_lkobma5tha
isExtra: false
isNew: false
operator: TEMPORAL_RANGE
sqlExpression: null
subject: emission_time
- clause: WHERE
comparator: registered
datasourceWarning: false
expressionType: SIMPLE
filterOptionName: filter_pj8co397y3o_0b0rs0sh42o
isExtra: false
isNew: false
operator: ==
operatorId: EQUALS
sqlExpression: null
subject: enrollment_status
conditional_formatting: []
extra_form_data: {}
force_timestamp_formatting: false
header_font_size: 0.5
metric:
aggregate: null
column: null
datasourceWarning: false
expressionType: SQL
hasCustomLabel: true
label: Enrolled Users
optionName: metric_rv2dquu9awp_95ed6x24fj
sqlExpression: count(distinct actor_id)
subheader: ''
subheader_font_size: 0.2
time_format: smart_date
viz_type: big_number_total
y_axis_format: ',d'
query_context:
datasource:
id: 73
type: table
force: false
form_data:
adhoc_filters:
- clause: WHERE
comparator: No filter
datasourceWarning: false
expressionType: SIMPLE
filterOptionName: filter_twlv610ia5j_lkobma5tha
isExtra: false
isNew: false
operator: TEMPORAL_RANGE
sqlExpression: null
subject: emission_time
- clause: WHERE
comparator: registered
datasourceWarning: false
expressionType: SIMPLE
filterOptionName: filter_pj8co397y3o_0b0rs0sh42o
isExtra: false
isNew: false
operator: ==
operatorId: EQUALS
sqlExpression: null
subject: enrollment_status
conditional_formatting: []
dashboards:
- 6046
datasource: 73__table
extra_form_data: {}
force: false
force_timestamp_formatting: false
header_font_size: 0.5
metric:
aggregate: null
column: null
datasourceWarning: false
expressionType: SQL
hasCustomLabel: true
label: Enrolled Users
optionName: metric_rv2dquu9awp_95ed6x24fj
sqlExpression: count(distinct actor_id)
result_format: json
result_type: full
slice_id: 3001
subheader: ''
subheader_font_size: 0.2
time_format: smart_date
viz_type: big_number_total
y_axis_format: ',d'
queries:
- annotation_layers: []
applied_time_extras: {}
columns: []
custom_form_data: {}
custom_params: {}
extras:
having: ''
where: ''
filters:
- col: emission_time
op: TEMPORAL_RANGE
val: No filter
- col: enrollment_status
op: ==
val: registered
metrics:
- aggregate: null
column: null
datasourceWarning: false
expressionType: SQL
hasCustomLabel: true
label: Enrolled Users
optionName: metric_rv2dquu9awp_95ed6x24fj
sqlExpression: count(distinct actor_id)
order_desc: true
series_limit: 0
url_params: {}
result_format: json
result_type: full
slice_name: Currently Registered Users
uuid: 15620ae6-5452-4a11-bc7e-56c7e5987a4d
version: 1.0.0
viz_type: big_number_total
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ _file_name: Event_Activity.yaml
cache_timeout: null
certification_details: null
certified_by: null
dataset_uuid: 6c54112c-1e2f-4c5f-8511-320fd9e180ac
dataset_uuid: 15d44a63-4a18-5497-8b24-fa8959afff3a
description: null
params:
adhoc_filters:
Expand Down

This file was deleted.

Loading

0 comments on commit 17a05e8

Please sign in to comment.