You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For the mobile replay player, we currently eagerly load all videos for the replay. This means we can potentially be loading ~720 videos at once, which is not ideal. Instead we want to intelligently load videos that are right before and right after the current time in the player. Since our "go back" button is 10 seconds, we might want to load 15 or 20 seconds before and after the current playing segment.
The text was updated successfully, but these errors were encountered:
- Optimize loading of videos for mobile replays
- This PR modifies the `_videos` array by converting it into a
dictionary that maps segment index to the video element. This makes it
easier to only load videos when requested, rather than preloading them
all at once.
- Also fixes a bug where `currentIndex` was consistently undefined in
the `createVideo` function, but it looks like it's working as expected
now
- Closes#67430
For the mobile replay player, we currently eagerly load all videos for the replay. This means we can potentially be loading ~720 videos at once, which is not ideal. Instead we want to intelligently load videos that are right before and right after the current time in the player. Since our "go back" button is 10 seconds, we might want to load 15 or 20 seconds before and after the current playing segment.
The text was updated successfully, but these errors were encountered: