Perfil de cor embutido na imagem

Patrick_David · Março 3, 2016, 5:36pm

Não tenho certeza se isso apareceu em outro lugar (os termos de busca, perfil de cor incorporado, retornam resultados não relacionados.. ).

Eu administro um fórum de fotografia de Software Livre em discuss.pixls.us. Como você pode imaginar, temos usuários muito exigentes quanto à forma como suas imagens são exibidas.

Recentemente, nos deparamos com um problema em um tópico sobre gerenciamento de cores:

Um dos fotógrafos notou que a funcionalidade de buscar imagens vinculadas e armazená-las localmente parece remover as informações de cor das imagens. Embora isso possa ser aceitável para economizar espaço na maioria dos fóruns, para um fórum de fotografia isso é um pouco doloroso.

Para ilustrar, esta imagem possui um perfil de cor incorporado rotacionado. Ela deveria mostrar (Rot = Vermelho, Grün = Verde, Blau = Azul):

A imagem fonte que incorporei na postagem acima pode ser encontrada aqui:

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

Aqui está a mesma imagem, enviada diretamente ao fórum:

O processamento de imagem está sendo feito com o ImageMagick? Se sim, há alguma consideração em adicionar uma opção para converter imagens enviadas para sRGB e marcá-las como tal antes de salvar?

(Eu ficaria feliz em investigar mais se alguém puder me apontar as partes relevantes do código que lidam com a conversão de imagem ao fazer o upload?)

codinghorror · Março 4, 2016, 2:53am

Sure check with @zogstrip.

Patrick_David · Março 4, 2016, 5:07pm

@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 · Março 4, 2016, 6:17pm

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 · Março 7, 2016, 10:43pm

Tested fix:

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

sam · Julho 18, 2018, 5:50am

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

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 · Julho 24, 2018, 3:10pm

Probably :comments is what we want there?

tgxworld · Julho 25, 2018, 12:20am

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 · Julho 25, 2018, 12:46am

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.

Tópico		Respostas	Visualizações
Image uploads - color profiles dropped Support uploads	5	685	23 de Setembro de 2025
Stripping image metadata when composer media optimization is enabled? Support	6	684	15 de Julho de 2021
No strip image metadata works for PNG but not JPG Bug	10	1026	24 de Maio de 2022
Uploading image makes it black (not darker!) Bug	14	3814	4 de Abril de 2015
Site setting for "strip image metadata" Announcements	3	1786	3 de Setembro de 2017

Perfil de cor embutido na imagem

Tópicos relacionados