Perfil de color incrustado en la imagen

Patrick_David · 3 Marzo, 2016 17:36

No estoy seguro de si esto apareció en otro lugar (los términos de búsqueda, perfil de color incrustado, devuelven resultados no asociados.. :).)

Dirijo un foro de fotografía de Software Libre en discuss.pixls.us. Como pueden imaginar, tenemos usuarios muy exigentes sobre cómo se muestran sus imágenes.

Recientemente nos encontramos con un problema en un hilo sobre gestión del color:

Uno de los fotógrafos notó que la funcionalidad de obtener imágenes enlazadas y almacenarlas localmente parece eliminar la información de color de las imágenes. Aunque esto puede ser aceptable para ahorrar espacio en la mayoría de los foros, para un foro de fotografía es bastante molesto.

Para ilustrarlo, esta imagen tiene un perfil de color incrustado rotado. Debería mostrar (Rot = Rojo, Grün = Verde, Blau = Azul):

La imagen original que incrusté en el mensaje anterior se puede encontrar aquí:

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

Aquí está la misma imagen, subida directamente al foro:

¿Se está realizando el procesamiento de imágenes con ImageMagick? De ser así, ¿se ha considerado la posibilidad de añadir una opción para convertir las imágenes subidas a sRGB y etiquetarlas como tal antes de guardarlas?

(Estaría encantado de investigar más a fondo si alguien pudiera indicarme las partes relevantes del código que gestionan la conversión de imágenes al subirlas).

codinghorror · 4 Marzo, 2016 02:53

Sure check with @zogstrip.

Patrick_David · 4 Marzo, 2016 17:07

@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 · 4 Marzo, 2016 18:17

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

github.com/discourse/discourse

app/models/optimized_image.rb

main


      
              SiteSetting.ImageQuality.image_preview_jpg_quality,
            )
          opts = opts.merge(quality: target_quality) if target_quality
          opts = opts.merge(upload_id: upload.id)
          
          # special case, when "resizing" vectors we simply copy
          if extension == ".svg"
            FileUtils.cp(original_path, temp_path)
            resized = true
          elsif opts[:crop]

darix · 7 Marzo, 2016 22:43

Tested fix:

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

sam · 18 Julio, 2018 05:50

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 · 24 Julio, 2018 11:49

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 )

github.com/discourse/discourse

lib/file_helper.rb

c3b681165


      
              # GLOBAL
              timeout: 15,
              skip_missing_workers: true,
              # PNG
              optipng: { level: 2, strip: SiteSetting.strip_image_metadata },
              advpng: false,
              pngcrush: false,
              pngout: false,
              pngquant: false,
              # JPG
              jpegoptim: { strip: SiteSetting.strip_image_metadata ? "all" : "none" },
              jpegtran: false,
              jpegrecompress: false,
            ).optimize_image!(filename)
          end
          
          private
          
          def self.images
            @@images ||= Set.new %w{jpg jpeg png gif tif tiff bmp svg webp ico}
          end

codinghorror · 24 Julio, 2018 15:10

Probably :comments is what we want there?

tgxworld · 25 Julio, 2018 00:20

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 · 25 Julio, 2018 00:46

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.

Tema		Respuestas	Vistas
Image uploads - color profiles dropped Support uploads	5	685	23 Septiembre 2025
Stripping image metadata when composer media optimization is enabled? Support	6	684	15 Julio 2021
No strip image metadata works for PNG but not JPG Bug	10	1026	24 Mayo 2022
Uploading image makes it black (not darker!) Bug	14	3814	4 Abril 2015
Site setting for "strip image metadata" Announcements	3	1786	3 Septiembre 2017

Perfil de color incrustado en la imagen

Temas relacionados