Profilo colore incorporato nell'immagine

Patrick_David · 3 Marzo 2016, 5:36pm

Non sono sicuro se questo sia già apparso altrove (i termini di ricerca, profilo colore incorporato, restituiscono risultati non correlati.. ).

Gestisco un forum di fotografia su software libero su discuss.pixls.us. Come puoi immaginare, abbiamo utenti molto esigenti su come vengono visualizzate le loro immagini.

Recentemente abbiamo riscontrato un problema in una discussione sulla gestione del colore:

Uno dei fotografi ha notato che la funzionalità di recupero delle immagini collegate e il loro salvataggio locale sembra rimuovere le informazioni sul colore dalle immagini. Sebbene questo possa andare bene per risparmiare spazio nella maggior parte dei forum, per un forum di fotografia è piuttosto doloroso.

Per illustrare, questa immagine ha un profilo colore incorporato ruotato. Dovrebbe mostrare (Rot = Rosso, Grün = Verde, Blau = Blu):

L’immagine sorgente che ho incorporato nel post sopra si trova qui:

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

Ecco la stessa immagine, caricata direttamente sul forum:

Il processo di elaborazione delle immagini viene effettuato con ImageMagick? In tal caso, è stato considerato l’aggiunta di un’opzione per convertire le immagini caricate in sRGB e contrassegnarle come tali prima del salvataggio?

(Sarei felice di indagare ulteriormente se qualcuno potesse indicarmi le parti di codice pertinenti che gestiscono la conversione delle immagini al momento del caricamento?)

codinghorror · 4 Marzo 2016, 2:53am

Sure check with @zogstrip.

Patrick_David · 4 Marzo 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 · 4 Marzo 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 · 7 Marzo 2016, 10:43pm

Tested fix:

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

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

Probably :comments is what we want there?

tgxworld · 25 Luglio 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 · 25 Luglio 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.

Argomento		Risposte	Visualizzazioni
Image uploads - color profiles dropped Support uploads	5	685	Settembre 23, 2025
Stripping image metadata when composer media optimization is enabled? Support	6	684	Luglio 15, 2021
No strip image metadata works for PNG but not JPG Bug	10	1026	Maggio 24, 2022
Uploading image makes it black (not darker!) Bug	14	3814	Aprile 4, 2015
Site setting for "strip image metadata" Announcements	3	1786	Settembre 3, 2017

Profilo colore incorporato nell'immagine

Argomenti correlati