-
Notifications
You must be signed in to change notification settings - Fork 325
Fix infinite loop in VkAudio.search_iter #403
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
Conversation
…udio.convert_m3u8_links Восстановлена работа функции VkAudio.get_audio_by_id
…ок на обложну трека (пустую строку, если обложек нет).
… 50 дублировались аудиозаписи, если у пользователя их больше 50
…o) VkAudio переписаны с использованием PHP бэкэнда - al_audio.php Метод search_iter больше не принимает параметр offset, поскольку "акт" (метод) section из al_audio.php не поддерживает смещение. Для получения подгрзуки результатов поиска используется метод load_catalog_section. Параметр count метода search был увеличен с 50 до 100. Функция scrap_data была разделена на три функции: scrap_ids, scrap_ids_from_html и scrap_tracks Была добавлена проверка наличия всех необходимых хэшей аудиозаписей.
…и у пользователя более 2000 аудиозаписей. Функция scrap_tracks теперь генератор, поскольку при большом количестве аудиозаписей она задерживала вывод результатов.
Метод get_audio_by_id возвращает полную информацию об аудиозаписи.
+функции get_popular_iter(), get_news_iter()( популярные и новинки) +оффсеты для search_iter(), get_popular_iter(), get_news_iter() +защита от пустого response 'playlist' в search_iter()
Уменьшил обьем кода для оффсетов
|
Если кто-то затестит, то можно мержнуть |
|
@python273 промежуточный результат такой, у меня в районе 3к треков, нашел 1955 по первому запросу, ссылки собирает, собрал в файл и выкачал .mp3 файлы, правда на мой взгляд надо ставить - возможность сколько за раз считывать у get_iter, так как delay большой у scrap_tracks мб @qwertyadrian и @dashedman подскажут |
|
@Enziferum ты имеешь в виду добавить методу get - аргументы count и offset? |
|
кто-нибудб затестите |

fixes #391
перенесено #402