Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/dim self paced pd activity #140

Merged
merged 33 commits into from
Aug 16, 2024
Merged

Feature/dim self paced pd activity #140

merged 33 commits into from
Aug 16, 2024

Conversation

allison-code-dot-org
Copy link
Collaborator

@allison-code-dot-org allison-code-dot-org commented Jul 29, 2024 •
edited
Loading

Description

final changes to dim_self_paced_pd_activity, ready for release

also creates unreleased (waiting for more clear guidelines on course implementation names) model, dim_self_paced_pd_analysis, in order to analyze self paced pd activity on the teacher/course grain. This model can answer questions like:

  • How many teachers who do self-paced go on to teach that course?
  • How far did teachers get in the self-paced curriculum on average?

Links

Jira ticket(s): DATAOPS-887

nataliazm99
nataliazm99 requested changes Aug 15, 2024
View reviewed changes
Copy link
Contributor

@nataliazm99 nataliazm99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some edits to the definition of slef-paced scripts, to make it more evergreen.
However, the definition of the course to be implemented is still manual and not perfect right now as some new courses don't have a clear student-facing parallel

dbt/models/marts/teachers/dim_self_paced_pd_activity.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_activity.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_activity.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_activity.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_activity.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql Outdated Show resolved Hide resolved
dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql Outdated Show resolved Hide resolved
allison-code-dot-org and others added 25 commits August 15, 2024 15:28
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
0e3629e 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
8a73031 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
ad35627 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
ed1451b 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org
Merge branch 'main' of https://github.com/code-dot-org/analytics into…
c037701 
… feature/dim_self_paced_pd_activity
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
cad3fc9 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
265db93 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
b17e2ed 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
3a0df15 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_activity.sql
85b64c5 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
672ac07 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
3f2293f 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
cf35686 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
a06b450 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
25ce74b 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
c55a487 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
258f24e 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
42542f6 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
d9c2d8c 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
2298d2a 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
d333fb1 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
3f0332b 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
ca8f8f9 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
5c78d2e 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
38feb4f 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
allison-code-dot-org and others added 4 commits August 16, 2024 13:57
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
9fa65b4 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
d6cda48 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org @nataliazm99
Update dbt/models/marts/teachers/dim_self_paced_pd_analysis.sql
34cf2a7 
Co-authored-by: Natalia Zacipa Mejia <98911841+nataliazm99@users.noreply.github.com>
@allison-code-dot-org
final changes to self paced pd activity
32aed3a
@allison-code-dot-org allison-code-dot-org merged commit f6ab2df into main Aug 16, 2024
1 check was pending
@allison-code-dot-org allison-code-dot-org deleted the feature/dim_self_paced_pd_activity branch August 16, 2024 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nataliazm99 nataliazm99 nataliazm99 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@allison-code-dot-org @nataliazm99