awxkee / jxl-coder

JPEG XL (JXL) Decoder/Encoder library for Android
Apache License 2.0
46 stars 5 forks source link

Скукожило #10

Closed T8RIN closed 8 months ago

T8RIN commented 8 months ago

Screenshot_20240107-200459.png

А что с jxl кодером стало (справа его работа), почему так кукожить он начал... Раньше точно все норм было, с какой версии это пошло не знаю, но сейчас стоит 1.5.7

awxkee commented 8 months ago

Какие это настройки? Я переделывал как-то уровни кач-ва, скорее всего твои настройки качества где-то не сходятся с новыми

T8RIN commented 8 months ago

image

T8RIN commented 8 months ago

на 1.5.5 так же чет

awxkee commented 8 months ago

Дай картинки которые жмешь, я не видел такого, может в картинке дело

T8RIN commented 8 months ago

Screenshot_20240107-191800.png

T8RIN commented 8 months ago

может в декодинге проблема?

image

T8RIN commented 8 months ago

Этот катмул ром не может так корявить?

Чекнул, вроде не в нем дело

awxkee commented 8 months ago

Я тут обновляю как раз jxl-coder, и там вообще в скейле был маленький баг, может из-за него

T8RIN commented 8 months ago

Возможно вполне, это ж мне скейлер тож надо будет обновить?

awxkee commented 8 months ago

Да, я посмотрю и скажу точно что это

T8RIN commented 8 months ago

Окей, спасибо :)

awxkee commented 8 months ago

Возможно вполне, это ж мне скейлер тож надо будет обновить?

Я посмотрел скейл, пофикшенный, после его применения на уменьшении картинки деградация примерно совпадает с андроидовским, может ты просто не применяешь правильно скейл в варианте не JXL? ( и еще Catmull Rom и все кубические сплайны сами по себе плохо работают на уменьшение )

Картинка сжатая моим скейлем с Catmull Rom в 3 раза,

Screenshot_20240107_174100_JXLCoder

Картинка сжатая андроид скейлом в 3 раза, Screenshot_20240107_174217_JXLCoder

Как по мне так Catmull Rom даже лучше справился

T8RIN commented 8 months ago

Не , к скейлеру вопросов нет вроде, а вот jxl странно делает...

awxkee commented 8 months ago

Я посмотрел кстати твой коммит, ты там путаешь, BSpline у меня был это бикубический сплайн, а Cubic - это обычные кубические сплайны

awxkee commented 8 months ago

Не , к скейлеру вопросов нет вроде, а вот jxl странно делает...

вот 2 изображения выше, одно я сжал в 3 раза JXL, 2 я сжал в 3 раза андроидом, я не вижу особой разницы

T8RIN commented 8 months ago

Я посмотрел кстати твой коммит, ты там путаешь, BSpline у меня был это бикубический сплайн, а Cubic - это обычные кубические сплайны

А, поменяю тогда местами

T8RIN commented 8 months ago

Не , к скейлеру вопросов нет вроде, а вот jxl странно делает...

вот 2 изображения выше, одно я сжал в 3 раза JXL, 2 я сжал в 3 раза андроидом, я не вижу особой разницы

Хм, вроде все норм я там показываю, щас загружу конвертнутый, я там даже не меняю размер а оно корявит

T8RIN commented 8 months ago

Screenshot_20240107-204909.png

T8RIN commented 8 months ago

ResizedImage_2024-01-07_20-51-01_1916.jpg

Этот jxl ни вставить ни выбрать нормально, поддержка околонулевая в системе, вот переименовал в jpg

T8RIN commented 8 months ago

Получилось по ссылке его открыть, Screenshot_20240107-205555.png

Тут никаких косяков не видно, может в декодере реально проблема

awxkee commented 8 months ago

Глянь какие у тебя dstWidth. + dstHeight передаются в декодер

T8RIN commented 8 months ago

В декодер коил передает, так что я не знаю

awxkee commented 8 months ago

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

T8RIN commented 8 months ago

Щас попробую

T8RIN commented 8 months ago

image

T8RIN commented 8 months ago

Screenshot_20240107-211507.png

Он с любой картинкой это вытворяет

Оригинал ResizedImage_2024-01-07_21-09-21_2030.png

T8RIN commented 8 months ago

Так, ну хотя не с каждой, ток при уменьшении

T8RIN commented 8 months ago

Но странно, в авифе ж все норм

awxkee commented 8 months ago

Так, ну хотя не с каждой, ток при уменьшении

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

awxkee commented 8 months ago

Так, ну хотя не с каждой, ток при уменьшении

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

в авиф билинейный скейл из libheif, я тут видимо забыл катмул ром убрать, не помню зачем я его поставил, но видимо сейчас тоже будет билинейный по дефолту и тут

T8RIN commented 8 months ago

Окей, а фиксы в XScaler'е будут?

awxkee commented 8 months ago

Окей, а фиксы в XScaler'е будут?

Да

T8RIN commented 8 months ago

image ааэаэ

awxkee commented 8 months ago

оно работает скопируй все правильно )

T8RIN commented 8 months ago

буковка к буковке скопировал

T8RIN commented 8 months ago

image

T8RIN commented 8 months ago

ааа, походу либы еще надо скопировать по новой

awxkee commented 8 months ago

да, hwy я обновил с libjxl

awxkee commented 8 months ago

1.6.0 версии попробуй как там со скейлом будет, я просмотрел кучу вариантов, вроде аномалий быть не должно

T8RIN commented 8 months ago

Окей

T8RIN commented 8 months ago

image

теперь ваще не работает

awxkee commented 8 months ago

Хм, это та же картинка?

T8RIN commented 8 months ago

ваще на любой падает

T8RIN commented 8 months ago

image

T8RIN commented 8 months ago

а скейлер работает норм, ток джксл пытаюсь сделать сразу креш через секунду

awxkee commented 8 months ago

Ок, буду смотреть, видимо что-то пошло не так

T8RIN commented 8 months ago

Буду ждать :)

awxkee commented 8 months ago

А билинейный скейл правилньо работает на твоем телефоне? Запусти на несольких картинках сразу

T8RIN commented 8 months ago

Опа, вылетает

T8RIN commented 8 months ago

А ты кста не можешь там try catch поставить, чтобы если падало, то java exception выбрасывала, нативные я ловить не могу, чтобы прога не падала, а прост креш активити показывалась