Total Commander

Flasher писал(а):По описание Number этим и является. Нет?

Не совсем, это функция преобразования, а я имел в виду тип колонки. Сейчас есть строка и целое, с плавающей точкой нет. Правильнее всего будет добавить этот тип.

Flasher писал(а):Ты имеешь в виду доступно в wlx? ОК. Но нам же нужно разделять их для видео и аудио. LonerD уже просит на видео.

Я про то и говорю, что информация по библиотеке в разных файлах бывает в разных контекстах - General, Video и Audio. Вроде информация как правило не пересекается, т.е. если есть в одном контексте, в других ничего нет, но это не точно. То, что источников понадобится целых три я уже понял... Надо посмотреть, можно ли (и имеет ли смысл) сделать из них одну колонку. Или придется и колонок делать три.

LonerD писал(а):Небольшое пожелание - добавить в xml-файл настройки

Ну по ширине/высоте выше написано, источники уже есть, нужно ли делать колонки надо подумать. Ширина иногда могла бы быть полезной.

По библиотеке кодирования тоже выше.

По "настройкам кодирования" не думаю, что эта простыня значений будет интересна среднестатистическому пользователю. Более того, как ее вообще использовать в wdx при такой длине? Даже в wlx я под нее делал функцию для деления на несколько строк. Так что скорее всего нет.

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

Loopback писал(а):Правильнее всего будет добавить этот тип.

Конечно. А ТС автоматически добавляет сравнительные опции для числовых типов, или в код плагина надо доп. функции вносить?

Loopback писал(а):Вроде информация как правило не пересекается, т.е. если есть в одном контексте, в других ничего нет

Не знал.

А можно пример, когда в General есть то, чего нет в остальных двух?

Loopback писал(а):По "настройкам кодирования" не думаю, что эта простыня значений будет интересна среднестатистическому пользователю.

Согласен. Этот вариант я рассматривал, когда составлял список, и сразу от его предложения отказался. С другой стороны "Все аудио/видео одной строкой" ты почему-то не чураешься:

Loopback писал(а):Пользы особой и нет (хотя некоторые пользуются), это в общем-то больше пример использования скриптов в колонках.

Loopback писал(а):Вообще у меня уже иногда появляется желание сделать второй файл примера, какой-нибудь extended.

Я об этом с начала обсуждения думал на примере TCFS2.Core.ini/TCFS2.ini (там лишний sample не требуется, просто не трогаем с Core), но потом, сопоставив разницу в правке ini/xml и результирующих конфликтов/тормозов, что-то передумалось.

Loopback писал(а): По "настройкам кодирования" не думаю, что эта простыня значений будет интересна

Среднестатистическому примерно 90-95% информации плагина/библиотеки не будет интересно

Loopback писал(а):Более того, как ее вообще использовать в wdx при такой длине?

Полностью она и не влезает. Но по начальным данным (ref, deblock, subme...) можно делать более-менее объективные выводы о качестве энкода.

Loopback писал(а):Даже в wlx я под нее делал функцию для деления на несколько строк.

Спасибо. Так намного удобней. Можно даже ещё чуть больше строк сделать (чтобы в режиме QuickView на мониторах 1920х1080 текст помещался на экране).
А с автоматическим переносом рядов (как реализовано в утилите MediaInfo Lite, к примеру) так ничего и не получилось?

LonerD писал(а):Среднестатистическому примерно 90-95% информации плагина/библиотеки не будет интересно

Такому вообще плагин не будет интересен, если уж на то пошло.

LonerD писал(а):Полностью она и не влезает.

В подсказку скорее влезет. В частности, комментарий больше нужен именно в ней. Но это не в поддержку, а просто к сведению.

Loopback
В конфиге и MediaInfo.lng в Portarait нужно a убрать.

Flasher писал(а):А ТС автоматически добавляет сравнительные опции для числовых типов

Да, поэтому смысл в числовых типах есть.

Flasher писал(а):А можно пример, когда в General есть то, чего нет в остальных двух?

Не, что-то я не то увидел

Посмотрел сейчас, всё зависит от формата. Например в mkv в General информация о контейнере, в Video о кодеке, а в Audio ничего. В mp4 бывает только информация в Video, не всегда, в других контекстах ничего. В аудиоформатах, во flac есть в Audio, в mp3 в General и Video одно и то же. Ну и так далее.

Короче говоря, нужны все три.

Flasher писал(а):С другой стороны "Все аудио/видео одной строкой" ты почему-то не чураешься

Ну я писал, откуда оно появилось. Можно и убрать, мне не жалко. Примеры есть и в других местах.

Flasher писал(а): но потом, сопоставив разницу в правке ini/xml и результирующих конфликтов/тормозов, что-то передумалось.

Технически оно было бы не так сложно, если бы колонки и режимы просмотра просто сваливались в кучу, но тут важен порядок. Ну и конфликтов при ручной правке не избежать, конечно.

LonerD писал(а):Среднестатистическому примерно 90-95% информации плагина/библиотеки не будет интересно

Всё верно, поэтому задача определить эти 5%, которые будут полезны большинству. Остальные без проблем добавят нужное.

LonerD писал(а):Можно даже ещё чуть больше строк сделать

В коде есть вызов функции FormatLibrarySettings(v, 8 ), число - это количество параметров в строке. Можно ставить меньше.

LonerD писал(а):А с автоматическим переносом рядов (как реализовано в утилите MediaInfo Lite, к примеру) так ничего и не получилось?

Даже скачал, посмотрел

Там просто Edit-контрол, здесь список. Теоретически, сделать перенос в пределах строки можно, но я не вижу большого смысла в таком усложнении. Из минусов добавится большая высота строки (как у обложки), что неудобно при листании. Но и ее максимальной высоты в 255 пикселей для такой простыни может и не хватить (встречал файлы у которых при настройках по умолчанию разбивается на ~20 строк), сейчас такой проблемы не возникает.

Flasher писал(а):В конфиге и MediaInfo.lng в Portarait нужно a убрать.

Спасибо, поправлю.

Loopback писал(а):Можно и убрать, мне не жалко.

Если никто категорично и аргументированно не воспротивится, убирай.

Loopback писал(а):Ну и конфликтов при ручной правке не избежать, конечно.

О них в т. ч. и речь.

Loopback писал(а):Всё верно, поэтому задача определить эти 5%, которые будут полезны большинству.

Как же? Тогда бы разговор шёл об урезании, а не расширении. Тут важно найти золотую середину для потенциальных пользователей не абы какого медиа-плагина, а именно на базе MediaInfo с её спецификой.

С остальным ясно. С русской справкой помощь нужна?

Loopback писал(а):"Все аудио/видео одной строкой"... Можно и убрать, мне не жалко.

А это уже обидно.
"All audio as string" использую в одном из вариантов колонок с информацией о видеофайлах чтобы не плодить лишние колонки для каждого пункта (в данном случае важнее инфо по видеоряду) и во всплывающей подсказке. На мой взгляд стоит наоборот чуть расширить выводимую информацию - добавить сюда же количество потоков и языки.
"All video as string" - во всплывающей подсказке (и битрейт лучше в KBps выводить, а не округлять до Mb)

LonerD писал(а):А это уже обидно.

Лучше какие-то значения в строку через запятую или всё же нормально именнованные в столбик? По мне, второе куда удобней.

LonerD писал(а):чтобы не плодить лишние колонки для каждого пункта

Так и не плоди. Объединяй их как твой душе угодно в любой последовательности в одну колонку, хоть через запятую, хоть через точку с запятой, хоть через вертикальную черту и т. д.

Если правильно понял, "All audio as string" / "All video as string" формируются из данных, которые плагин _уже_ получил: убирание не скажется на производительности заметно. А раз кем-то используется, то тем более смысла убирать особо и нет.
Плюс это хороший пример использования скриптового языка.

Skif_off писал(а): А раз кем-то используется, то тем более смысла убирать особо и нет.

То же самое можно сказать про убираемые "Параметры формата". Логика в том, что любую из этих строк можно сформировать правкой в диалоге без xml.
К тому же толку от сбиваемой строчным списком, который можно было выдумать каким угодно, сортировки тут особой нет.

Flasher писал(а):По мне, второе куда удобней.

Удобство - вопрос спорный. Мне, например, основные параметры аудиодорожек гораздо удобнее читать именно так, короткой строкой (ещё пример - в компактном режиме просмотра видео). В wdx я этим полем не пользуюсь, тут это реально делалось как пример.

Но раз это кому-то нужно - пусть теперь остается. Убирать что-то всегда сложнее, так как всегда найдется кто-нибудь, кто этим пользовался. Потому и спор о нужности полей уже на три страницы, совсем не хочется надобавлять лишнего, чтобы потом болезненно это убирать

Flasher писал(а):Объединяй их как твой душе угодно в любой последовательности в одну колонку

Главный минус стандартного подхода - пустые строки (или лишние разделители) в случае отсутствия значения. Скриптовый вариант это позволяет это исключить.

Loopback писал(а):Главный минус стандартного подхода - пустые строки (или лишние разделители) в случае отсутствия значения.

Никаких пустых подстрок (мы говорим про однострочники) и лишних разделителей не будет, если их записывать в кавычках внутри тега до/после его обозначения:

Код: Выделить всё

[=tcmediainfo.Видеоформат][=", "tcmediainfo.Битрейт видео.MBps" Мбит/с"][=", "tcmediainfo.Размер кадра][=", "tcmediainfo.Частота кадров" к/с"]

Flasher
У меня отображает, если нет значений(возможно значения пустые):

Lisabon
Что-то я не понял, куда смотреть на скриншоте?

Flasher
На всплывающую подсказку к файлу.

Total Commander

[wlx/wdx] TCMediaInfo