The chroma upscaling code is missing.

[ilyakurdyukov]

Since the time that you forked my repository - I have added chroma upscaling code to better match jpeg2png, which also has such a feature. And I implemented it myself, but in a different way.

[zvezdochiot]

Hi @ilyakurdyukov

Good. I'll see.

[ilyakurdyukov]

Notice, that I have jpegqs-mini.c which have all the features (but enabled/disabled at compile time) except SIMD, and easier to read.

[zvezdochiot]

Hi @ilyakurdyukov .

I've already done a visual test for quality = 50. The question is blur?

[ilyakurdyukov]

Напиши по русски, я не понимаю что ты имеешь в виду.

[zvezdochiot]

Да мне и самому неудобно, но приходится. Пока я понял, что в моей версии осталось простое обрезание дробной составляющей, в твоей учитывается половина квантователя. Из-за этого у меня дополнительное размытие и определённая потеря оттенка.

Визуальный тест производил на Лене с quality = 50 стандартным cjpeg.

[ilyakurdyukov]

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

Возможно будет проще просто взять jpegqs-mini.c за основу, и перенести на него твои изменения.

[zvezdochiot]

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

[ilyakurdyukov]

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

[zvezdochiot]

Илья, чтобы не тянуть резину, могу тебя заInvokeть, но незадача. И тебе есть чем заняться и заниматься "дублем" кода - это кал, и я не собираюсь заниматься этим кодом в ближайшее время (мозг занят другим проектом из другой Organization). Стоит отложить данную тему на месяцок. Ежели прям горит, то могу и заInvokeть.

[ilyakurdyukov]

Мне есть чем заняться. Просто я хотел заметить что код твоего форка заметно устарел от моего и, возможно, тебе тоже нужны эти улучшения.