mitodl / mitodl/ol-data-platform
Migrate `instructor_module_report` to use dimensional layer (blocked by mart migration)
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 51
- Forks
- 6
- Avg merge
- 2d 19h
- Merged PRs (30d)
- 113
Description
Epic: #2072
File: src/ol_dbt/models/reporting/instructor_module_report.sql
Problem: References marts__combined_course_enrollment_detail and the organization_administration_report reporting model (which itself bypasses). Blocked by migration of the mart and the reporting model.
Note: This issue is blocked by the migration of the upstream mart model(s) it references. It should be addressed after those mart migrations are complete.
Acceptance criteria:
- Model references only
dimensional/, mart, or reporting models (all of which use dimensional) - No
{{ ref('int__*') }}or{{ ref('stg__*') }}calls remain in this model or its upstream mart dependencies - Model compiles and passes dbt tests
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
After the upstream mart and reporting-model migrations are complete, start with src/ol_dbt/models/reporting/instructor_module_report.sql and inspect its references to marts__combined_course_enrollment_detail and organization_administration_report. Confirm that upstream dependencies no longer bypass the dimensional layer, then run dbt compilation and tests. Done means the model and its upstream mart dependencies contain no int__* or stg__* refs and pass dbt tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- sql
- Domain
- data-engineering
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100