Bug 14422

Summary: [Fix 21] simple-scan - Added support for B&W scans (for exams)
Product: [ROSA-based products] ROSA Fresh Reporter: VictorR2007 <victorr2007>
Component: Packages from MainAssignee: ROSA Linux Bugs <bugs>
Status: CONFIRMED --- QA Contact: ROSA Linux Bugs <bugs>
Severity: normal    
Priority: Normal CC: irgaliev01, m.novosyolov, pribytkovav02, sulmpx60, temap, v.potapov, victorr2007, vladislav.tarakanov
Version: All   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Platform: --- ROSA Vulnerability identifier:
RPM Package: ISO-related:
Bad POT generating: Upstream:
Attachments: ЧЕК-ЛИСТ

Description VictorR2007 2024-04-15 19:39:28 MSK
Собрано в пул реквест https://abf.io/import/simple-scan/pull_requests/4

Для экзаменов требуется поддержка чёрно-белого сканирования (сейчас simple-scan позволяет сканировать либо цветное изображение, либо с градацией серого). Обсуждение по этой функции уже пару лет ведутся (https://gitlab.gnome.org/GNOME/simple-scan/-/issues/224). В коде на данный момент всё реализовано по функционалу, не хватало просто кнопки, решение о которой гномовские дизайнеры похоже всё никак не примут, и пары строк для с привязкой кнопки к действию. Чуть позже отправим PR с реализаицией этой фичи и с кнопкой, возможно примут в апстрим.

Попутно добавили запись недостающих флагов для TIF-файла. Из основных флагов: dpi по X и Y. Без этого некоторые редакторы некорректно принимают изображение. Также были проблемы с загрузкой этих файлов на сайт, связанный с этими экзаменами.

Сборки
https://abf.rosalinux.ru/build_lists/5022188
https://abf.rosalinux.ru/build_lists/5022189
https://abf.rosalinux.ru/build_lists/5022190
https://abf.rosalinux.ru/build_lists/5022191
https://abf.rosalinux.ru/build_lists/5022192
Comment 1 Artem Pribytkov 2024-04-17 17:29:40 MSK
Created attachment 6082 [details]
ЧЕК-ЛИСТ

Проведенные проверки описал в чек-листе.
Comment 2 Vladimir Potapov 2024-04-19 09:47:23 MSK
С точки  зрения интерфейса добавление в меню где выбор между текстом и изображением "черно-белый" - очень плохое решение.
А как выбрать "черно-белый" текст или черно-белое изображение?
Просится галочка в диалоге, где выбирается качество изображения т.к. "черно-белый" относится именно к качеству.
Comment 3 Vladimir Potapov 2024-04-19 09:51:49 MSK
Далее, в меню сохранения для всех видов написано (сжатый) или (без потери в качестве) и только для TIFF написано LZW. Это излишняя для пользователя информация, другого сжатия ж не предлагается а в тонкостях алгоритмов большинство не разбирается. 
Предлагаю заменить TIFF (сжатый LZW) на TIFF (без потери качества)
И да, не "без потери в качестве" а "без потери качества".
Comment 4 Vladislav Tarakanov 2024-04-19 10:31:40 MSK
(In reply to Vladimir Potapov from comment #2)
> А как выбрать "черно-белый" текст или черно-белое изображение?

Такого выбора в программе нет. Тоже не понравилось текущее сочетание, но решили не влезать в апстримовские переводы. Суть там сейчас такая: файктически simple-scan не разделяет сканирование текста или изображения. Выбор изображения выполняет цветное сканирование, выбор текста - с градацией серого. Лучший вариант в текущей ситуации, наверное, это замена стандартных переводов

(In reply to Vladimir Potapov from comment #3)
> Предлагаю заменить TIFF (сжатый LZW) на TIFF (без потери качества)

Странно, в патче обновили перевод, чтобы был просто TIFF, без меток про сжатие и качество. Из общего перечня такой вариант выбивается, из-за отсутствия скобок, но можно дописать по аналогии с WebP и JPEG, "(сжатый)".
Comment 5 Vladimir Potapov 2024-04-19 10:51:01 MSK
(In reply to Vladislav Tarakanov from comment #4)

> сжатие и качество. Из общего перечня такой вариант выбивается, из-за
> отсутствия скобок, но можно дописать по аналогии с WebP и JPEG, "(сжатый)".
(сжатый) как раз не надо, подразумевается что сжатый - это вариант с потерей качества.
Comment 6 Vladimir Potapov 2024-04-19 10:55:00 MSK
(In reply to Vladislav Tarakanov from comment #4)

> файктически simple-scan не разделяет сканирование текста или изображения.
> Выбор изображения выполняет цветное сканирование, выбор текста - с градацией
> серого. 
Они отличаются не только цветным-серым, там для них еще и разрешение задается в настройках. И это правильно - человек выбирает что сканировать а система предлагает оптимальные настройки для этого. 

Я предлагаю галочку черно-белого сканирования добавить в настройках качества, убрав кнопку черно-белого сканирования, тогда логика интерфейса сохранится.
Comment 7 Vladislav Tarakanov 2024-04-19 11:12:13 MSK
(In reply to Vladimir Potapov from comment #5)
> (сжатый) как раз не надо, подразумевается что сжатый - это вариант с потерей
> качества.

TIFF сейчас идёт обязательно с сжатием, либо с LZW, либо с CCITT 4, на выбор пользователя (последнее только в ЧБ доступно).

(In reply to Vladimir Potapov from comment #6)
> Я предлагаю галочку черно-белого сканирования добавить в настройках
> качества, убрав кнопку черно-белого сканирования, тогда логика интерфейса
> сохранится.

Интерфейс там сейчас уже не соотностится с тем, что происходит на практике. Текст и изображение это названия для серого и цветного сканирования. Для них выбираются разные DPI, да, но мы просто не добавляли возможность корректировать этот параметр для ЧБ сканов, там по умолчанию 300.

"Изображение" сейчас сканирует в 3 канала (RGB), по 8 бит. "Текст" - 1 канал с то ли 2 битами, то ли 8, не помню уже точно. "Чёрно-белый" - 1 канал, 1 бит
Comment 8 Vladimir Potapov 2024-04-19 11:49:00 MSK
Ох. Предлагаемая вами схема полностью ломает логику, simple-scan потому и simple, что там все просто было - сканируем либо текст, либо изображение. Всё.
Для профессиональной точной настройки есть другие программы, тот же xsane
Comment 9 Vladimir Potapov 2024-04-19 11:55:57 MSK
Вот, отсканировал одну страницу как текст, вторую черно-белую (гравюра, например). И выбора сжатия при сохранении нет.
Предлагаемый интерфейс не рабочий полностью - он не позволяет для ч/б ни разрешение настроить, ни метод сохранения для больших файлов.
Comment 10 Vladislav Tarakanov 2024-04-19 12:01:33 MSK
(In reply to Vladimir Potapov from comment #9)
> Вот, отсканировал одну страницу как текст, вторую черно-белую (гравюра,
> например). И выбора сжатия при сохранении нет.

Выбора сжатия и не должно быть, если есть хоть 1 скан не в ЧБ. CCITT 4 не применим для серых или цветных изображений, только для чёрно-белых. Так как в вариантах с двумя разновидностями сканов (ЧБ и серым, в данном случае) одна из страниц не будет поддерживать CCITT, то и выбрать его нельзя
Comment 11 Vladislav Tarakanov 2024-04-19 12:16:44 MSK
(In reply to Vladimir Potapov from comment #8)
> Ох. Предлагаемая вами схема полностью ломает логику, simple-scan потому и
> simple, что там все просто было - сканируем либо текст, либо изображение.
> Всё.
> Для профессиональной точной настройки есть другие программы, тот же xsane

В этом и проблема, что названия сейчас в Simple-scan немного странные, не совсем отображают то, что делается в действительности. В режиме "Текст" не обязательно сканировать именно текст. Никаких проблем не возникнет, если отсканировать в этом режиме изображение. Да и никаких особых плюсов после сканирования текста в режиме "Текст" нет.

Видимо в апстриме сделано было предположение, что все картинки будут сканировать цветными, а все тексты серыми, но, на мой взгляд, интуитивно не сразу можно понять, что для получения серого скана необходимо выбрать режим "Текст", особенно если на листке есть и текст и какая-нибудь картинка.
Comment 12 Vladimir Potapov 2024-04-19 12:53:07 MSK
(In reply to Vladislav Tarakanov from comment #10)
> Выбора сжатия и не должно быть, если есть хоть 1 скан не в ЧБ. CCITT 4 не
> применим для серых или цветных изображений, только для чёрно-белых. Так как
> в вариантах с двумя разновидностями сканов (ЧБ и серым, в данном случае)
> одна из страниц не будет поддерживать CCITT, то и выбрать его нельзя
Но зачем тогда вообще выбор-то? Если ч/б автоматически сохранять в CCITT, иначе в lzw. Оба алгоритма без потери качества, потому так и написать.
Comment 13 Vladislav Tarakanov 2024-04-19 12:59:46 MSK
(In reply to Vladimir Potapov from comment #12)
> Но зачем тогда вообще выбор-то? Если ч/б автоматически сохранять в CCITT,
> иначе в lzw. Оба алгоритма без потери качества, потому так и написать.

В таком случае, необходимо куда-нибудь в интерфейс вписать отображение текущего сжатия. Так как LZW допустимо и для ЧБ сканов, пользователя надо как-то оповестить, какое именно сжатие используется.
Comment 14 VictorR2007 2024-05-01 16:30:19 MSK
Продолжение в новом баге
https://bugzilla.rosalinux.ru/show_bug.cgi?id=14443