-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fixed pagination playlist #1640
base: main
Are you sure you want to change the base?
Conversation
оно не работает, больше 100 песен не отображаются в плейлисте, проверяй на 200-300 песнях |
Исправил, также все-таки вернул FJP, но теперь к каждому плейлисту есть "аффинити" на конкретный воркер в FJP, т е несколько плейлистов могут загружаться параллельно, но никак сами треки |
Вообще, оно как-то странно работает "надокачку" плейлиста. Я модифицировал таблицу треков, теперь Для "виртуальных" плейлистов поведение пока остается предыдущим - без пагинации |
я честно говоря не помню как работает - догрузка у самого вк, но наверное при стуке гет каталог оно просто аддает к текущему надо чекать, показывать по 100 песен и давать порциями |
Он сам все это делает, я делал свою пагинацию, для этого пришлось протащить свой (ну и extends не AbstractList, а ArrayList т к по какой-то причине решили указать какая именно реализация должна быть в AudioGetPlaylist.c Но от этой затеи я отказался по той причине, что: после того, как я указал audio_count в Playlist, о магия, он начал указывать разные offset на каждой прогрузке Это было больно, разбирать декомпилированный клиент, где все - это a,b, a, b с разными параметрами |
audio_count всегда был 0 потому что раньше была проблема что если отдавать количество песен из кеша, то тогда при подсчёте оно дохло у многих, пришлось костыли делать шо бы грузилось всё сразу и без лишних стуков в базу :(((( я соберу версию в бета тему, так что залетай в туда ибо на гитхабе гемор обсуждать |
Стоп, не совсем догоняю, бета тема - куда? |
# Conflicts: # app/src/main/java/ru/vtosters/lite/music/cache/db/Database.java # app/src/main/java/ru/vtosters/lite/music/converter/ID3Tagger.java # app/src/main/java/ru/vtosters/lite/music/downloader/Mp3Downloader.java
Исправил баг с пагинацией оффлайн плейлиста. Из-за того, что не было точного указания кол-во треков в плейлисте она не работала.
Загрузка плейлиста теперь происходит асинхронно
Пул для загрузки заменен на пул из одного потока, порядок скачивания треков для плейлиста важен, ни один пул не может гарантировать правильный порядок загрузки