LumaVU - визуализатор люминесцентного индикатора уровня сигнала

[HardWareMan]

На текущий момент AIMP инициализирует плагины по мере надобности, я так полагаю, что LumaVU в данном примере НИ РАЗУ не инициализировался, и упал при попытке выгрузить dll-ку. Возможно, что-то не учитывает, что плагин не был инициализирован.

Вот это уже информация. С какой версии данное поведение у АИМПа? А вообще это умнО: вызывать финализацию без инициализаии и при этом никому не сказать (или я пропустил это?).

[Artem]

Вот это уже информация. С какой версии данное поведение у АИМПа? А вообще это умнО: вызывать финализацию без инициализаии и при этом никому не сказать (или я пропустил это?).

Вообще загрузка по мере надобности у АИМПа была всегда. Т.е. финализация у плагина вызывается, даже если он не был инициализирован?

[Artem]

вызывать финализацию без инициализаии и при этом никому не сказать (или я пропустил это?).

финализация у плагина вызывается, даже если он не был инициализирован?

Проверил сам, ваша догадка ложна. Финализация плагина НЕ вызывается, если он не был инициализирован.

P.S. Вообще мне не очень понятны ваши выпады в наш адрес

[HardWareMan]

ОК, плагин использует следующие вызовы:

Code: [Select]

    function AuthorName: PWideChar; stdcall;
    function PluginName: PWideChar; stdcall;
    function PluginInfo: PWideChar; stdcall;
    function PluginFlags: DWORD; stdcall;
    function Initialize: BOOL; stdcall;
    procedure Deinitialize; stdcall;
    procedure DisplayClick(X, Y: Integer); stdcall;
    procedure DisplayRender(DC: HDC; AData: PVisualData); stdcall;
    procedure DisplayResize(AWidth, AHeight: Integer); stdcall;
    procedure HandleMenuClick(Sender: TObject);
    procedure MakeLangFile;
Если по вашим словам Initialize и Deinitialize не вызываются у пассивных плагинов,  функции AuthorName, PluginName, PluginInfo и PluginFlags не используют каких-либо  объектов, DisplayClick и HandleMenuClick физически не могут быть вызваны (кликаем то по крестику системного меню), MakeLangFile вызывается только в Initialize, который по вашим словам не вызывается. Остаются DisplayRender и DisplayResize. В первый я слабо верю - нет смысла вызывать рендер при закрытии, а вот изменение размера запросто, ибо на подобные грабли уже наступал в другом проекте. Но и в нем только присваивание текущих значений к локальным переменным и он не вызывает проблем если вызывался AIMP_QueryVisual, где создается главный объект через вызов TLumaVUPlugin.Create. Ведь он вызывался для неактивных плагинов?

Никаких выпадов. Народ хочет разобраться. (с) Потому как я не знаю механику интерфейса плагина визуализации вашего плеера. И, вне зависимости от разницы между GDI и GDI+,  если плеер поддерживает без особых проблем старый плагин (не крашится в виду неверного интерфейса), то предполагается, что и механика интерфейса не изменилась. Однако на практике мы видим другое. Поясните мне пожалуйста по поводу вышеуказанных функций и процедур в AIMP3. Или ткните носом где читать.

[HardWareMan]

Ну что-ж, вот и правда.

Я повесил вывод мессаджбокса на каждой функции. И порядок их вызова следующий (при неактивном плагине). Запускаю:

Все, действительно нет инициализации. Жму крестик:

Т.е., вы считаете это нормальным поведение плеера? Теперь я вам вот что скажу. Моему плагину требуется время на инициализацию, так как он в отдельном одноразовом потоке загружает данные для отображения экранов. Раньше я их грузил в основном потоке и все жаловались на подвисание плеера при выборе моего плагина. Потоку требуется время и шустрый вызов инита и следом деинита вызывают отрыв потока загрузки от объектов и, собственно, ошибку. Я, конечно, поправил эту несправедливость и теперь плаг не крашит AIMP3, если плаг не выбран для отображения. А вам следует подумать над поведением своего плеера.


PS В данной копии плагина просто исправлен баг, данная версия развиваться более не будет. Все запрошенные плюшки, хранение данных в том числе, будут реализованы в новой версии на новом движке, которая сейчас потихоньку пишется.

[Artem]

Да, вы правы, плеер на мгновение активирует следующую визуализацию по списку при закрытии, это происходит из-за того, что текущий плагин выгружается. Да, это не очень красиво, поправлю в ближайшем билде. Однако, запускать поток и не контролировать время его жизни - тоже плохой жест, финализация плагина придумана именно для того, чтобы плагин мог все за собой подчистить перед тем, как его выгрузят.

[Artem]

Offtop: Я сейчас работаю над новым API, у вас есть какие-нибудь пожелания по содержанию / юзабилити?

[MisterExciter]

Offtop: Я сейчас работаю над новым API, у вас есть какие-нибудь пожелания по содержанию / юзабилити?

Артём, сделай возможность получить цвет фона канваса для вывода визуалки. Чтобы можно было для индикатора получить "родной" фон плеера. Тогда можно будет сделать визуалку практически в стиле "а ля пандемик аналог метр".

[HardWareMan]

Да, вы правы, плеер на мгновение активирует следующую визуализацию по списку при закрытии, это происходит из-за того, что текущий плагин выгружается. Да, это не очень красиво, поправлю в ближайшем билде. Однако, запускать поток и не контролировать время его жизни - тоже плохой жест, финализация плагина придумана именно для того, чтобы плагин мог все за собой подчистить перед тем, как его выгрузят

Ну я, как обычно, положился на плеер. Так что счет 1:1. Ну и напомню, что в AIMP2 проблем не было.

Сейчас у меня только один вопрос, жизненно важный для моего плагина: откуда брать исходные данные об уровне сигнала. Отсюда вопрос: по какому алгоритму формируется встроенный уровень? Ну и как поступает WAVE сигнал: непрерывные кусочки или небольшие несвязанные кусочки?

[Artem]

Ну я, как обычно, положился на плеер. Так что счет 1:1. Ну и напомню, что в AIMP2 проблем не было.

Багу можно повторить, если два раза кликнуть по галке - загрузить / выгрузить. Или в процессе пролистывания визуализаций.

Сейчас у меня только один вопрос, жизненно важный для моего плагина: откуда брать исходные данные об уровне сигнала. Отсюда вопрос: по какому алгоритму формируется встроенный уровень? Ну и как поступает WAVE сигнал: непрерывные кусочки или небольшие несвязанные кусочки?

Структура VisualData уже имеет посчитанные уровни для обоих каналов, стандартный плагин использует именно их. WAVE поступает несвязанными кусочками, т.к. плагин работает с частотой всего 25 fps.

[HardWareMan]

Багу можно повторить, если два раза кликнуть по галке - загрузить / выгрузить. Или в процессе пролистывания визуализаций.

Это уже извращение. Тем не менее, я вставил в деинит ожидание закрытия потока, что дает небольшой лаг при закрытии. Баг может вылезти только если придут 2 инита подряд без деинита между ними. Ладно, в штатном режиме работает, перекантуемся до новой версии.

Структура VisualData уже имеет посчитанные уровни для обоих каналов, стандартный плагин использует именно их. WAVE поступает несвязанными кусочками, т.к. плагин работает с частотой всего 25 fps.

Понятно. А как считаются уровни? По какому алгоритму? Средневзвешанные? Квадратичные или просто пиковые? Анализируется весь непрерывный поток или так же кусочки с 25fps? Просто иногда замечаю, что некоторые пики ударных пропадают.

PS Можно данное поведение плеера оставить (переинит при закрытии), если передавать плагину флаг "Close_Query", чтобы он знал что можно делать а что нельзя.

[Artem]

Тем не менее, я вставил в деинит ожидание закрытия потока,

Видимо, это не сработало по каким-то причинам.

что дает небольшой лаг при закрытии.

Ничего страшного

Баг может вылезти только если придут 2 инита подряд без деинита между ними.

Это невозможно

А как считаются уровни? По какому алгоритму? Средневзвешанные? Квадратичные или просто пиковые? Анализируется весь непрерывный поток или так же кусочки с 25fps? Просто иногда замечаю, что некоторые пики ударных пропадают.

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

PS Можно данное поведение плеера оставить (переинит при закрытии), если передавать плагину флаг "Close_Query", чтобы он знал что можно делать а что нельзя.

В смысле? Что значит можно и нельзя? Плагин должен корректно обрабатывать команды, что ему посылает ядро.

[Soolo]

Кстати, о корректности отображения полосок и соответствия слышимой музыке.
Иногда один из каналов "зашкаливает" на несколько сек, хотя музыка идёт в это время явно тихая, без уровня 0 дБ.
Даже при остановке музыки - полоска остаётся "зашкаленой".
Стоит настройка "логарифмическая шкала"
Ещё хотел узнать, где плагин хранит настройки

[HardWareMan]

В смысле? Что значит можно и нельзя? Плагин должен корректно обрабатывать команды, что ему посылает ядро.

Ладно, забей.

Кстати, о корректности отображения полосок и соответствия слышимой музыке.
Иногда один из каналов "зашкаливает" на несколько сек, хотя музыка идёт в это время явно тихая, без уровня 0 дБ.
Даже при остановке музыки - полоска остаётся "зашкаленой".
Стоит настройка "логарифмическая шкала"

Эта проблема была озвучена ранее (несколько страниц назад). Можно и ее подкорректировать на скорую руку. Проявляется она, почему-то, только на х64 системах.

[Soolo]

Эта проблема была озвучена ранее (несколько страниц назад). Можно и ее подкорректировать на скорую руку. Проявляется она, почему-то, только на х64 системах.

Сори, не следмл раньше за этим плагином. Да, у меня как раз x64. Впрочем, объяснение самое простое. Некоторые типы данных зависят от разрадности системы, к примеру, Real или Integer в Дельфях.