Profil couleur intégré dans l'image

[Patrick_David (Pat David)]

Je ne sais pas si cela a déjà été signalé ailleurs (les termes de recherche, profil de couleur intégré, retournent des résultats non associés.. ).

Je gère un forum sur le logiciel libre dédié à la photographie sur discuss.pixls.us. Comme vous pouvez l’imaginer, nous avons des utilisateurs très exigeants sur la façon dont leurs images sont affichées.

Récemment, nous avons rencontré un problème dans un fil de discussion sur la gestion des couleurs :

Correct way to work with color managed applications?

L’un des photographes a remarqué que la fonctionnalité de récupération des images liées et de leur stockage local semble supprimer les informations de couleur des images. Bien que cela puisse être acceptable pour économiser de l’espace sur la plupart des forums, pour un forum de photographie, c’est un peu douloureux.

Pour illustrer, cette image possède un profil de couleur intégré rotatif. Elle devrait afficher (Rouge = Rouge, Vert = Vert, Bleu = Bleu) :

L’image source que j’ai intégrée dans le message ci-dessus se trouve ici :

https://houz.org/tmp/farbkreis.jpg

Voici la même image, téléchargée directement sur le forum :

Le traitement de l’image est-il effectué avec ImageMagick ? Si oui, a-t-on envisagé d’ajouter une option pour convertir les images téléchargées en sRGB et les étiqueter comme telles avant l’enregistrement ?

(Je serais heureux d’approfondir l’enquête si quelqu’un pouvait m’indiquer les parties pertinentes du code qui gèrent la conversion des images lors du téléchargement.)

[codinghorror (Jeff Atwood)]

Sure check with @zogstrip.

[Patrick_David (Pat David)]

@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.

[zogstrip]

@Patrick_David

is the conversion being done using imagemagick?

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

app/models/optimized_image.rb

[darix]

Tested fix:

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

[sam (Sam Saffron)]

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?

[tgxworld (Alan Tan)]

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 )

lib/file_helper.rb

[codinghorror (Jeff Atwood)]

Probably :comments is what we want there?

[tgxworld (Alan Tan)]

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.

[sam (Sam Saffron)]

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.