retropassdev / RetroPass

Retro Pass is a simple frontend for emulators running on Xbox.
GNU General Public License v2.0
105 stars 42 forks source link

Crashes upon succes video views #41

Closed John-Birdflu closed 4 months ago

John-Birdflu commented 9 months ago

Seems to crash the app when several videos are viewed in succession.

retropassdev commented 9 months ago

As discussed, this seems to be connected to the size of the videos. High quality game videos might take to much memory resources and combined with a huge library of games and preloaded thumbnails, it might be just enough to crash the app. Xbox dev mode allows "only" 1GB of memory per app. This is plenty for regular apps, but for a frontend where there's a lot of dynamic loading and preloading, it might be less than required in some cases.

Also as John mentioned, replacing videos with lower quality videos fixed the issue. However, if you have a huge library, even just browsing through platforms might eventually crash the app due to a large number of preloaded thumbnails.

Retropass does not do any kind of memory management, since it's kind of an edge case, but might look into that in the future.

retropassdev commented 4 months ago

Closing for now.