Встроенный цветовой профиль изображения

Внести вклад Ошибка
1

Не уверен, что это где-то уже обсуждалось (поисковые запросы, такие как встроенный цветовой профиль, выдают нерелевантные результаты.. :slight_smile: ).

Я веду форум о свободном программном обеспечении для фотографии на discuss.pixls.us. Как вы можете представить, у нас есть пользователи, которые очень придирчивы к тому, как отображаются их изображения.

Недавно мы столкнулись с проблемой в теме о управлении цветом:

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

Для наглядности: на этом изображении вращается встроенный цветовой профиль. Оно должно отображаться так (Rot = Красный, Grün = Зелёный, Blau = Синий):

farbkreis
farbkreis800×800 82.9 KB

Исходное изображение, которое я вставил в пост выше, можно найти здесь:

А вот то же изображение, загруженное напрямую на форум:

farbkreis.jpg800×800 176 KB

Обработка изображений выполняется с помощью ImageMagick? Если да, то рассматривалась ли возможность добавления опции для конвертации загружаемых изображений в sRGB и маркировки их как таковых перед сохранением?

(Я с радостью готов углубиться в исследование, если кто-то укажет мне на соответствующие части кода, отвечающие за конвертацию изображений при загрузке?)

5 лайков
2

Sure check with @zogstrip.

3

@zogstrip - is the conversion being done using imagemagick? If so, would you mind pointing me to the relevant section?

It appears that we may only need an option to convert any images to sRGB if they have a different profile embedded, and then either strip the profile or specify that the image is now sRGB.

4

We do the resizing with convert but then the thumbnail is optimised using optipng.

You will find everything you need to know in OptimizedImage and more specifically

3 лайка
5

Tested fix:

https://github.com/discourse/discourse/pull/4064

3 лайка
6

This still does not look fixed to me… I just tried with the image on the OP.

@tgxworld as part of your image magick 7 upgrade can you test this?

3 лайка
8

This is actually a result of ImageOptim and not ImageMagick. We probably don’t want to be stripping the ICC profile here?

  • :strip — List of extra markers to strip: :comments , :exif , :iptc , :icc or :all (defaults to :all )
3 лайка
9

Probably :comments is what we want there?

1 лайк
10

I was thinking about this and this seems very specific to a photography forum. If they want to keep all the metadata/color profile, they can just set SiteSetting.strip_image_metadata to false. For majority of the images, keeping the exif or iptc metadata isn’t going to be very useful.

2 лайка
11

Interesting @Patrick_David, so this is not really a bug… it is a feature request to allow for fidelity when stripping exif tags (or simply misunderstanding an existing feature)

I am going to close this for now, but feel free to open a new topic if you find setting strip_image_metadata to false unacceptable.

5 лайков