search

openedx-unsupported / edx-analytics-pipeline

GNU Affero General Public License v3.0
91 stars 116 forks source link

Remove unnecessary course modes from enrollment cohort report #812

Closed zacharis278 closed 4 years ago

zacharis278 commented 4 years ago

Cleans up report by removing enrollment counts for course modes where no enrollments exist. e.g. masters in an xseries program.

MST-79

codecov[bot] commented 4 years ago

Codecov Report

Merging #812 into master will increase coverage by 0.01%. The diff coverage is 93.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #812      +/-   ##
==========================================
+ Coverage   74.38%   74.39%   +0.01%     
==========================================
  Files         211      211              
  Lines       24415    24435      +20     
==========================================
+ Hits        18161    18179      +18     
- Misses       6254     6256       +2
Impacted Files Coverage Δ
edx/analytics/tasks/programs/program_reports.py 93.44% <92.85%> (-0.28%) :arrow_down:
...ytics/tasks/programs/tests/test_program_reports.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a75e26d...4879bad. Read the comment docs.