Repository navigation
Add watch time, completion and trends to the data top list - #10
Open
bluebird42 wants to merge 1 commit into
Open
bluebird42 wants to merge 1 commit into
bluebird42 wants to merge 1 commit into
Conversation
The top list follows the selected period and shows watch time, average completion and a views trend per video next to views. Views, watch time and completion columns sort the list. The ranking query moves into Analytics.Overview so it shares the overview's definition of a view. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The top list only showed views for this year, so a video with many short views looked the same as one people watch to the end.
The top list now follows the period switch and shows, per video:
?sort=watch_timeor?sort=completion; views by default).The ranking query moves from
Analytics.space_top_videos/2(added in #7) toAnalytics.Overview.top_videos/4, so the list uses the same definition of a view as the overview. Deleted videos are still left out.Stacked on #9.
Top list for the last 30 days (synthetic data):
Validation:
Analytics.Overviewtests cover ranking, watch time, completion, the per-period trend and leaving out views outside the period.mix compile --warnings-as-errors,mix format --check-formatted, Credo andgit diff --checkpassed.StoragePolicyTestfailures also occur onmainin my local setup.No migration is needed.