Open aizenbit opened 8 years ago
Domerk
commented
8 years ago При запуске прога разворачивается не на весь экран, и при этом при диагонали 11,6 дюймов ещё и не влезает на него полностью. Это выглядит больно:
Мб стоит разворачивать его во весь экран. Я для этого использую this->setWindowState(Qt::WindowMaximized); в конструкторе главного окна, если что.
aizenbit
commented
8 years ago Хм, нет, это не решает проблему влезания, если открывать большие картинки, когда прога уже развёрнута во весь экран. Мне кажется, правильнее будет сделать прокрутку.
Domerk
commented
8 years ago Плюс один к скроллу или чему-то подобному. При открытии картинки она не масштабируется под размер окна, а наоборот, окно растягивается. Привести в нормальный размер его можно, свернув прогу в окно и снова развернув на весь экран. Но это всё равно успевает причинить боль:
Domerk
commented
8 years ago Хьюстон, я там выше пишу про то, что прога не влазит в экран сразу при запуске, тк дефолтные размеры окна больше, чем можно себе позволить с диагональю 11,6 и меньше. Их можно просто уменьшит, конешн, но имхо лучше сразу на весь экран разворачивать. В дополнение к масштабированию и/или скроллингу картинок.
aizenbit
commented
8 years ago А, понял. Тогда согласен. Правда, вряд ли сегодня у кого-то экран меньше, чем 11,6 дюймовый, так что просто уменьшить окно так, чтобы оно туда влезало - нормальный вариант.
Domerk
commented
8 years ago У самого Димы на нетбуке меньше, насколько я помню х) Кстати у него минимальный размер окна задан как 943x799 (или что-то около того). Это, конечно, уже придирки, но не иметь возможности сделать окно такого размера, какого хочется - не очень круто.
dive155
commented
8 years ago Отсутствует скролл картинки.
При открытии картинки она не масштабируется под размер окна, а наоборот, окно растягивается.
Всё, что касается проблем с большими картинками будет фикситься после написания пояснительной записки, а может и после защиты. Пока что прога будет корректно работать только с маленькими картинками.
При запуске прога разворачивается не на весь экран, и при этом при диагонали 11,6 дюймов ещё и не влезает на него полностью.
Насчет дефолтного размера окна действительно есть косяк, на одолженном ноуте тоже не влезло. Уменьшу.
Утечки памяти в MainWindow.
Можно поподробнее?
Поначалу показалось немного не очевидно, что цветовой тон/насыщенность не накладывается на изменение цветового баланса. Может быть, вынести их в разные страницы ToolBox'а?
Хмм они действительно не накладываются друг на друга. Выносить в отдельную вкладку будет муторно т.к. это целый новый класс, так что в дипломной версии оно останется как есть. Эти две коррекции можно применить последовательно если применить сначала первую, сохранить изменения и затем применить вторую.
Что-то непонятное при изменении насыщенности
Тут я пожалуй буду винить имплементацию HSL в кьюте. Ползунки элементарно меняют параметры HSL пикселя, с моей стороны там нет никаких сложных алгоритмов.
При слишком долгом выполнении фильтра программа зависает, прогресс-бар не меняет своё значение.
Хмм думаю я знаю как фиксить это.
Был ещё момент, когда в лог шёл спам сообщениями QColor::setHsv: HSV parameters out of range. Заметил их только когда дошёл до шумоподавления; воспроизвести не удалось.
Ну вот было бы неплохо воспроизвести. А визуально результат обработки соответствовал ожиданиям?
Привести в нормальный размер его можно, свернув прогу в окно и снова развернув на весь экран. Ты обнаружила растягивание картинки в проге в которой его никогда не было о_0
aizenbit
commented
8 years ago Всё, что касается проблем с большими картинками будет фикситься после написания пояснительной записки, а может и после защиты.
Ну да, конечно)
Можно поподробнее?
У тебя там в конструкторе выделяется куча памяти под классы, а деструктор почти пустой, то есть выделенная память не освобождается при закрытии программы.
Тут я пожалуй буду винить имплементацию HSL в кьюте. Ползунки элементарно меняют параметры HSL пикселя, с моей стороны там нет никаких сложных алгоритмов.
Я сейчас пригляделся к картинке, и, кажется, проблема именно в ней. Слишком большое сжатие. Сорри)
Ну вот было бы неплохо воспроизвести.
Сейчас быстро пробежался по коду, и мне кажется, что проблема в инверсии. Там идёт изменение hue без какой-либо проверки на выход за допустимое значение. Правда, воспроизвести спам мне не удалось.
Ты обнаружила растягивание картинки в проге в которой его никогда не было о_0
Она не растягивается, а обрезается.
createDIB: CreateDIBSection failedQColor::setHsv: HSV parameters out of range. Заметил их только когда дошёл до шумоподавления; воспроизвести не удалось.