Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The chroma upscaling code is missing. #1

Open
ilyakurdyukov opened this issue Dec 26, 2021 · 10 comments
Open

The chroma upscaling code is missing. #1

ilyakurdyukov opened this issue Dec 26, 2021 · 10 comments
Labels
enhancement New feature or request

Comments

@ilyakurdyukov
Copy link

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.

@ilyakurdyukov ilyakurdyukov changed the title The chrome upscaling code is missing. The chroma upscaling code is missing. Dec 26, 2021
@zvezdochiot
Copy link
Member

Hi @ilyakurdyukov

Good. I'll see.

@zvezdochiot zvezdochiot added the enhancement New feature or request label Dec 26, 2021
@ilyakurdyukov
Copy link
Author

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
Copy link
Member

Hi @ilyakurdyukov .

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

@ilyakurdyukov
Copy link
Author

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

@zvezdochiot
Copy link
Member

zvezdochiot commented Dec 26, 2021

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

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

@ilyakurdyukov
Copy link
Author

ilyakurdyukov commented Dec 26, 2021

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

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

@zvezdochiot
Copy link
Member

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

@ilyakurdyukov
Copy link
Author

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

@zvezdochiot
Copy link
Member

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

@ilyakurdyukov
Copy link
Author

ilyakurdyukov commented Dec 26, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants