feat: adds models for video engagement reports (FC-0024) #6
Conversation
`video_plays` contains a record for each time a learner played a video. Can be used to generate reports of total plays and unique watchers `transcript_usage` contains a record for each time a transcript or closed caption was enabled. Can be used to generate reports for total transcription/caption usage as well as the number of unique learners who enabled transcription/closed captioning
|
Thanks for the pull request, @SoryRawyer! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
openedx-webhooks
added
the
open-source-contribution
|
Is this going to need to be refactored after the mv changes? |
|
It is, just a little bit. I'll update the PR to reflect the MV changes. |
The .github/workflows/self-assign-issue.yml workflow is missing or needs an update to stay in sync with the current standard for this workflow as defined in the `.github` repo of the `openedx` GitHub org.
The .github/workflows/add-depr-ticket-to-depr-board.yml workflow is missing or needs an update to stay in sync with the current standard for this workflow as defined in the `.github` repo of the `openedx` GitHub org.
The .github/workflows/commitlint.yml workflow is missing or needs an update to stay in sync with the current standard for this workflow as defined in the `.github` repo of the `openedx` GitHub org.
The .github/workflows/add-remove-label-on-comment.yml workflow is missing or needs an update to stay in sync with the current standard for this workflow as defined in the `.github` repo of the `openedx` GitHub org.
SoryRawyer
force-pushed
the
init_video_engagement_models
branch
from
8f4a0d4
to
55d5061
Compare
|
It doesn't look like my rebase worked as I'd expected, so I'm just going to close this PR and open a new one with the same changes. |
|
@SoryRawyer Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future. |
Adds two new models for issue #3 :
video_plays: contains a record for each time a learner played a video. Can be used to generate reports of total plays and unique watcherstranscript_usage: contains a record for each time a transcript or closed caption was enabled. Can be used to generate reports for total transcription/caption usage as well as the number of unique learners who enabled transcription/closed captioningFor each model,
org,course_id, andvideo_idare available as dimensions.Question for reviewers: is there a more robust way of deriving a video ID from the
object_id? This approach works for test data but uses a relatively naive approach and may not accurately capture the video ID from different environments.Note: this PR does not define the metrics available from these tables as they also need to be defined in Superset. I have Superset datasets (and an example chart) for each model and can export and share them if that's helpful.