Skip to content

feat: Final round of cleanup #159

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
Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

feat: Final round of cleanup #159

wants to merge 6 commits into from

Conversation

@saraburns1
Copy link
Contributor

@saraburns1 saraburns1 commented Sep 11, 2025 •
edited
Loading

  1. Remove unneeded macros (used only in 1 place)
  2. Convert views to MVs
  • dim_course_blocks
  • fact_navigation_completion
  • fact_pageview_engagement
  • fact_problem_engagement
  1. New dim_items_per_subsection mv instead of macro
  2. Combine most_recent_course_grade and most_recent_course_state
  3. New dim_problem_coursewide_avg to replace tutoraspects query
  4. New dim_subsection_performance to replace tutoraspects query
  5. Code standardization - table aliases on all joins
  6. Updated unit tests

@saraburns1 saraburns1 force-pushed the dbt_cleanup branch 3 times, most recently from 5a269fd to e3d456f Compare September 12, 2025 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@saraburns1