AIMP Forum

AIMP for Android => Ошибки и замечания / Bugs => Обработано / Processed => Topic started by: captain-nemo on February 11, 2022, 08:58:26

Title: [x] [1185] Приоритет обложки из интернета
Post by: captain-nemo on February 11, 2022, 08:58:26
в 1164 при воспроизведении радио имелся приоритет обложки из интернета, например, при выполнении списка воспроизведения

#EXTM3U
#EXTINF:-1 logo="file:///storage/emulated/0/logoradio/monte-carlo.jpg",Монте-Карло
https://str5.pcradio.ru/montecarlo_moscow-med


при наличии тэга от радиостанции показывалась обложка из интернета, при отсутствии тэга показывалась локальная картинка, хранящаяся на диске (например во время новостей).

в 1183 при выполнении того же списка, при наличии локальной картинки, безусловно показывается она, тэг радиостанции при этом игнорируется.
Если из списка исключить ссылку на локальную картинку,

#EXTM3U
#EXTINF:-1,Монте-Карло
https://str5.pcradio.ru/montecarlo_moscow-med

 то тэг выполняется, показывается обложка из интернета, но во время новостей или при отсутствии тэга показывается отсутствие обложки.

Устройства: Huawey P Smart 2019 Android 10; Teyes CC2+ Android 10
Title: Re: [1183] Приоритет обложки из интернета
Post by: Artem on February 13, 2022, 21:34:18
Это неправильно. Приоритет должен быть строго такой: теги > локальная обложка > сеть.
Именно такое поведение задокументировано и присутствует в AIMP for Windows.

P.S. Сейчас сравнивал код, и нашел-таки ошибку - это лишняя точка с запятой, которая блокировала преждевременный выход из функции.

P.P.S. Сценарий ваш понимаю, но если уж его поддерживать, то поддерживать нужно везде и на ПК, ни мобиле, а то иначе поведение разнится.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: captain-nemo on February 14, 2022, 14:46:34
[1184] Опробовал, все осталось по- прежнему. Приоритета обложки из интернета нет.
Title: Re: [1183] Приоритет обложки из интернета
Post by: Soolo on February 14, 2022, 14:56:29
Quote from: Artem on February 13, 2022, 21:34:18
P.P.S. Сценарий ваш понимаю, но если уж его поддерживать, то поддерживать нужно везде и на ПК, ни мобиле, а то иначе поведение разнится.
Для радио нужно своё поведение сделать
Т.к. там не один файл играет, а разные
По сути, ведь обложка не в файле, а прописана в теге URL
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Artem on February 14, 2022, 15:22:23
Quote from: Soolo on February 14, 2022, 14:56:29
Для радио нужно своё поведение сделать

Ты не можешь знать в общем случае, радио это или нет. Конечно, редко, кто слушает файлы через http/ftp, но все же бывает.
Не спорю, что придумывать надо, т.к. сценарий вполне понятный и логичный. Но, повторюсь, если делать - делать и для ПК, и для телефона.

Quote from: captain-nemo on February 14, 2022, 14:46:34
Опробовал, все осталось по- прежнему. Приоритета обложки из интернета нет.

Кажется, вы не прочитали мой ответ =(
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Soolo on February 14, 2022, 15:28:02
Quote from: Artem on February 14, 2022, 15:22:23
Не спорю, что придумывать надо, т.к. сценарий вполне понятный и логичный. Но, повторюсь, если делать - делать и для ПК, и для телефона.
Нужно, да.
Не вижу противоречий ни для Винды, ни для Андроида

Quote from: Artem on February 14, 2022, 15:22:23
Ты не можешь знать в общем случае, радио это или нет. Конечно, редко, кто слушает файлы через http/ftp, но все же бывает.
По идее, для таких файлов нужно считывать обложку именно из правильного тега файла, а не тега URL
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Artem on February 14, 2022, 16:44:27
Попробуйте:
https://disk.yandex.ru/d/1YCYrTHPUdsPFw
https://disk.yandex.ru/d/RzbfjMbuRadSnQ
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: captain-nemo on February 17, 2022, 09:05:50
Опробовал версию [1184] с Яндекс диска, которую вы выложили 16.02.22. Приоритет обложек работает здОрово, есть тег - обложка из инета, нет тега - локальная эмблема радиостанции. Спасибо!
Версию для Windows из предыдущего поста установил, обложки из интернета она показывает, а как установить отображение локальной обложки, пока не разобрался.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Artem on February 17, 2022, 09:32:38
Quote from: captain-nemo on February 17, 2022, 09:05:50
Версию для Windows из предыдущего поста установил, обложки из интернета она показывает, а как установить отображение локальной обложки, пока не разобрался.

Так же, как и для телефона
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: captain-nemo on February 17, 2022, 09:35:24
Видимо, неправильно прописываю путь...
#EXTM3U
#EXTINF:-1 logo="file:///D:\Дистрибутивы\Android\AIMP\Радио\logoradio\Iskatel.jpg",Искатель
https://iskatel.hostingradio.ru:8015/iskatel-128.mp3

 ;D Разобрался, слеш лишний...

#EXTM3U
#EXTINF:-1 logo="file://D:\Дистрибутивы\Android\AIMP\Радио\logoradio\Iskatel.jpg",Искатель
https://iskatel.hostingradio.ru:8015/iskatel-128.mp3
Но показывает только картинку с диска.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Artem on February 17, 2022, 09:53:09
Работает, только не для всех треков. Мобильная версия не валидирует поисковую выдачу - не проверяет соответствует обложка именно искомому треку или нет. ПКшная версия более совершенная и она-таки это проверяет, и если обложка не подходит - не показывает ее.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: captain-nemo on February 17, 2022, 10:50:18
Да, увидел. Некоторые показываются. Спасибо!
Большинство обложек показывает.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: drafted on February 23, 2022, 14:04:43
Quote from: Artem on February 13, 2022, 21:34:18
Приоритет должен быть строго такой: теги > локальная обложка > сеть.
1. Для меня, при прослушивании интернет-радио, предпочтительней отображение логотипа радиостанции (локального [и выбранного мной], прописанного в плейлисте). Всегда понятно, какую станцию слушаешь.

2. Большинство радиостанций, которые я слушал, "не передают обложки в потоке". Не знаю, как это реализовывается технически и есть ли какие-либо стандарты на этот счёт, потому проверяю "правильную" работу этой функции в плеере Neutron. При прочих равных, он отображает обложки в потоках только одной станции:
[Radio Paradise](https://radioparadise.com/player) (например, основной поток https://stream.radioparadise.com/aac-128). Были ещё одна-две станции, которые тоже это могли. Сейчас не вспомню.
Упомянутое выше Радио Монтекарло (https://str5.pcradio.ru/montecarlo_moscow-med) так не умеет.

3. В качестве компромисса в этом вопросе прошу сделать следующее:
Так как, в разделе настроек "Обложки альбомов", для того чтобы логотип отображался в принципе, должна быть активна настройка "Искать в интернете (Да)", предлагаю отдавать приоритет локальной обложке, прописанной в плейлисте радиостанций, при установлении значения настройки "Макс. размер для скачивания" равным "0"
Тогда, если пользователю хочется видеть обложки конкретных треков, звучащих на радиостанции, он устанавливает значение "Макс. размер для скачивания" большим "0"

P.S. Последняя версия, которая всегда отображала локальные логотипы, при настройках предлагаемых выше - 1183
Настройки отображения локального логотипа радиостанции
(https://i.ibb.co/yszLb1x/aimp-radiocover.png) (https://ibb.co/yszLb1x)
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Artem on February 23, 2022, 19:33:27
Quote from: Artem on February 14, 2022, 16:44:27
Попробуйте:
https://disk.yandex.ru/d/1YCYrTHPUdsPFw
https://disk.yandex.ru/d/RzbfjMbuRadSnQ
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: drafted on February 23, 2022, 20:06:45
 
Quote from: Artem on February 23, 2022, 19:33:27
Попробуйте
Ссылки уже не работают.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: coldun on February 23, 2022, 20:37:11
Quote from: drafted on February 23, 2022, 20:06:45
Ссылки уже не работают.
https://www.upload.ee СПАМ СПАМ/files/13912915/AIMP.rar.html
или
https://bit.ly/3vcyhT8
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: drafted on February 23, 2022, 20:45:14
Quote from: coldun on February 23, 2022, 20:37:11
https://bit.ly/3vcyhT8
В версиях 1184 и 1185 уже "сломан" алгоритм назначения обложек.
Я их пробовал.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: coldun on February 23, 2022, 21:18:33
@drafted
я уж не знаю насчёт сломано или нет, просто скинул файлы, на которые
Quote
Ссылки уже не работают.
(https://www.aimp.ru/forum/index.php?action=dlattach;topic=68509.0;attach=65650)
непонятно зачем Artem дал эти ссылки, файлы по которым он сам же и удалил... ???
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Artem on February 23, 2022, 23:19:32
Quote from: drafted on February 23, 2022, 14:04:43
P.S. Последняя версия, которая всегда отображала локальные логотипы, при настройках предлагаемых выше - 1183

Я не правильно прочитал текст, думал, вы говорите про то же, что и топик стартер. Оказывается, вы против!
П.С. Локальные логотипы продолжают показываться, но при условии что ни станция, ни discogs не имеют обложек для проигрываемого трека.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: drafted on February 24, 2022, 08:34:36
Quote from: Artem on February 23, 2022, 23:19:32
Оказывается, вы против!
Я к тому, что, если в версиях после 1183 вносились изменения в алгоритм отображения обложек, то была бы возможность изменять это поведение настройкой, предложенной мной в моём первом посте. Чтобы можно было использовать новые версии программы (не хотелось бы терять доступ к изменениям/исправлениям из-за остановки на версии 1183)
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Artem on February 24, 2022, 11:25:27
Давайте для начала разберемся, в чем именно в вашем случае проблема. А уже потом я приму решение, стоит ли городить целую опцию.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: drafted on February 24, 2022, 13:57:37
Quote from: Artem on February 24, 2022, 11:25:27
в чем именно в вашем случае проблема
Я использую
следующие
(https://i.ibb.co/yszLb1x/aimp-radiocover.png) (https://ibb.co/yszLb1x)
настройки отображения обложек альбомов/логотипов радиостанций.

До версии 3.22 (+/- какие-то билды), к радиостанциям (80+ станций) из плейлиста с прописанными локальными логотипами, последние отображались почти на всех. На 4-рёх, 5-ти станциях логотип либо не появлялся, либо периодически пропадал/появлялся.

После какого-то билда версии 3.22, с этими же настройками стали отображаться 100% логотипов. Т.е., все соответствующие локальные логотипы радиостанций, прописанные в плейлисте, отображались и не пропадали.

После билда 1183, версии 3.22, с указанными выше настройками ситуация следующая:
Часть логотипов отображается (в основном, станции разговорного жанра и некоторые музыкального), а часть - не отображается. Если установить настройку "Макс. размер для скачивания" большим "0", то, на тех станциях, на которых не отображались логотипы, начинают отборажаться обложки альбомов проигрываемого трека.

Получается, что я никак не могу добиться отображения логотипа радиостанции перманентно.
Поэтому, и предложил использовать настройку "Макс. размер для скачивания" равным "0" для установки приоритета локального логотипа, прописанного в плейлисте.
Title: Re: [+] [1183] Приоритет обложки из интернета
Post by: Artem on February 24, 2022, 15:51:20
Вопрос тут не в приоритете. Проблему я понял.
Title: Re: [1185] Приоритет обложки из интернета
Post by: Artem on February 25, 2022, 12:43:28
Вот, попробуйте:
https://disk.yandex.ru/d/ti6A5pj3drZ-sg
Title: Re: [x] [1185] Приоритет обложки из интернета
Post by: drafted on February 25, 2022, 21:13:48
Quote from: Artem on February 25, 2022, 12:43:28
попробуйте:
https://disk.yandex.ru/d/ti6A5pj3drZ-sg
При настройках, указанных мной выше (0КБ), ни одна обложка (локальная) не отображается.
Если установить 64КБ, разговорные и значительное кол-во других (например, тех, у которых нет тегов альбом/исполнитель) - не отображается. Остальные показывают обложку текущего трека.

На чём проверено
main 25/02/2022 21:14:22.601 D/App: v3.22 Beta, build 1186 (25.02.2022)
main 25/02/2022 21:14:22.605 D/App: motorola(Moto E (4) Plus, Android v7.1.1)
Title: Re: [x] [1185] Приоритет обложки из интернета
Post by: Artem on February 25, 2022, 21:29:17
Лог от программы пришлите, у меня все работает
Title: Re: [x] [1185] Приоритет обложки из интернета
Post by: drafted on February 26, 2022, 07:27:42
Quote from: Artem on February 25, 2022, 21:29:17
Лог от программы
Лог 1186
Title: Re: [x] [1185] Приоритет обложки из интернета
Post by: drafted on February 26, 2022, 07:59:08
Вернулся на билд 1183. Все локальные обложки отображаются.

Лог 1183
Title: Re: [x] [1185] Приоритет обложки из интернета
Post by: Artem on February 26, 2022, 09:28:45
Спасибо, вижу причину.
Вот, попробуйте теперь:
https://disk.yandex.ru/d/ti6A5pj3drZ-sg
Title: Re: [x] [1185] Приоритет обложки из интернета
Post by: captain-nemo on February 26, 2022, 10:27:04
[1186] по ссылке из предыдущего поста - все работает, как описано. android 10 huawei p smart 2019.
Title: Re: [x] [1185] Приоритет обложки из интернета
Post by: drafted on February 26, 2022, 10:47:47
Quote from: captain-nemo on February 26, 2022, 10:27:04
попробуйте теперь:
https://disk.yandex.ru/d/ti6A5pj3drZ-sg
Да, подтверждаю.
Теперь всё чётко.
Даже, если в реальном времени установить значение выше "0" (и начнут отображаться обложки треков), то, при последующем возврате значения на "0", текущий трек доигрывает с обложкой альбома, а к следующему треку применяется локальный логотип станции.
Спасибо!

Конечно, нужно ещё некоторое время потестировать...