search

appsembler / figures

Reporting and data retrieval app for Open edX
MIT License
44 stars 37 forks source link

Active Learners today inaccurate #54

Open johnbaldwin opened 6 years ago

johnbaldwin commented 6 years ago

Active learners today needs to filter on unique user ids for a given course on a given date. Currently the count is too high as it doesn't filter on unique user ids.

Ref: https://github.com/appsembler/figures/blob/0.1.3/figures/pipeline/course_daily_metrics.py#L67-L73

johnbaldwin commented 6 years ago

Found this while doing a code walkthrough for Maxi for Tahoe

johnbaldwin commented 6 years ago

PR https://github.com/appsembler/figures/pull/57 out for review on this