Разные мелочи

[aizenbit]

• Отсутствует скролл картинки. Так, при загрузке картинки с паровозиком элементы управления справа улетают в никуда. Впрочем, это справедливо для любого изображения с разрешением большим, чем разрешение экрана. Кроме того, программа подвисает при загрузке слишком больших изображений.
• Если развернуть прогу на весь экран, загрузить в неё паровозик и свернуть в окно, прога всегда вылетает, в лог при этом идёт createDIB: CreateDIBSection failed
• Утечки памяти в MainWindow.
• Поначалу показалось немного не очевидно, что цветовой тон/насыщенность не накладывается на изменение цветового баланса. Может быть, вынести их в разные страницы ToolBox'а?
• Что-то непонятное при изменении насыщенности.
• При слишком долгом выполнении фильтра программа зависает, прогресс-бар не меняет своё значение.
• Был ещё момент, когда в лог шёл спам сообщениями QColor::setHsv: HSV parameters out of range. Заметил их только когда дошёл до шумоподавления; воспроизвести не удалось.

[Domerk]

При запуске прога разворачивается не на весь экран, и при этом при диагонали 11,6 дюймов ещё и не влезает на него полностью. Это выглядит больно:

Мб стоит разворачивать его во весь экран. Я для этого использую this->setWindowState(Qt::WindowMaximized);
в конструкторе главного окна, если что.

[aizenbit]

Хм, нет, это не решает проблему влезания, если открывать большие картинки, когда прога уже развёрнута во весь экран. Мне кажется, правильнее будет сделать прокрутку.

[Domerk]

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

[Domerk]

Хьюстон, я там выше пишу про то, что прога не влазит в экран сразу при запуске, тк дефолтные размеры окна больше, чем можно себе позволить с диагональю 11,6 и меньше. Их можно просто уменьшит, конешн, но имхо лучше сразу на весь экран разворачивать. В дополнение к масштабированию и/или скроллингу картинок.

[aizenbit]

А, понял. Тогда согласен. Правда, вряд ли сегодня у кого-то экран меньше, чем 11,6 дюймовый, так что просто уменьшить окно так, чтобы оно туда влезало - нормальный вариант.

[Domerk]

У самого Димы на нетбуке меньше, насколько я помню х)
Кстати у него минимальный размер окна задан как 943x799 (или что-то около того). Это, конечно, уже придирки, но не иметь возможности сделать окно такого размера, какого хочется - не очень круто.

[dive155]

Отсутствует скролл картинки.

При открытии картинки она не масштабируется под размер окна, а наоборот, окно растягивается.

Всё, что касается проблем с большими картинками будет фикситься после написания пояснительной записки, а может и после защиты. Пока что прога будет корректно работать только с маленькими картинками.

При запуске прога разворачивается не на весь экран, и при этом при диагонали 11,6 дюймов ещё и не влезает на него полностью.

Насчет дефолтного размера окна действительно есть косяк, на одолженном ноуте тоже не влезло. Уменьшу.

Утечки памяти в MainWindow.

Можно поподробнее?

Поначалу показалось немного не очевидно, что цветовой тон/насыщенность не накладывается на изменение цветового баланса. Может быть, вынести их в разные страницы ToolBox'а?

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

Что-то непонятное при изменении насыщенности

Тут я пожалуй буду винить имплементацию HSL в кьюте. Ползунки элементарно меняют параметры HSL пикселя, с моей стороны там нет никаких сложных алгоритмов.

При слишком долгом выполнении фильтра программа зависает, прогресс-бар не меняет своё значение.

Хмм думаю я знаю как фиксить это.

Был ещё момент, когда в лог шёл спам сообщениями QColor::setHsv: HSV parameters out of range. Заметил их только когда дошёл до шумоподавления; воспроизвести не удалось.

Ну вот было бы неплохо воспроизвести.
А визуально результат обработки соответствовал ожиданиям?

Привести в нормальный размер его можно, свернув прогу в окно и снова развернув на весь экран.
Ты обнаружила растягивание картинки в проге в которой его никогда не было о_0

[aizenbit]

Всё, что касается проблем с большими картинками будет фикситься после написания пояснительной записки, а может и после защиты.

Ну да, конечно)

Можно поподробнее?

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

Тут я пожалуй буду винить имплементацию HSL в кьюте. Ползунки элементарно меняют параметры HSL пикселя, с моей стороны там нет никаких сложных алгоритмов.

Я сейчас пригляделся к картинке, и, кажется, проблема именно в ней. Слишком большое сжатие. Сорри)

Ну вот было бы неплохо воспроизвести.

Сейчас быстро пробежался по коду, и мне кажется, что проблема в инверсии. Там идёт изменение hue без какой-либо проверки на выход за допустимое значение. Правда, воспроизвести спам мне не удалось.

Ты обнаружила растягивание картинки в проге в которой его никогда не было о_0

Она не растягивается, а обрезается.