Eingebettetes Farbprofil im Bild

Patrick_David · 3. März 2016 um 17:36

Ich bin mir nicht sicher, ob das woanders schon einmal aufgetaucht ist (die Suchbegriffe color profile embedded liefern keine zusammenhängenden Ergebnisse.. ).

Ich leite ein Forum für Freie Software im Bereich Fotografie unter discuss.pixls.us. Wie Sie sich vorstellen können, haben wir Nutzer, die sehr genau darauf achten, wie ihre Bilder dargestellt werden.

Etwas, das uns kürzlich begegnet ist, waren Probleme in einem Thread zum Thema Farbmanagement:

Einer der Fotografen bemerkte, dass die Funktion zum Abrufen verlinkter Bilder und zum lokalen Speichern die Farbinformationen aus den Bildern zu entfernen scheint. Während dies auf den meisten Foren zur Speichereinsparung in Ordnung sein mag, ist es für ein Fotografie-Forum etwas schmerzhaft.

Zur Veranschaulichung: Dieses Bild hat ein gedrehtes eingebettetes Farbprofil. Es sollte so aussehen (Rot = Rot, Grün = Grün, Blau = Blau):

Das Quellobild, das ich in den obigen Beitrag eingebettet habe, finden Sie hier:

Hier ist dasselbe Bild, direkt auf das Forum hochgeladen:

Wird die Bildverarbeitung mit ImageMagick durchgeführt? Falls ja, gibt es Überlegungen, möglicherweise eine Option hinzuzufügen, um hochgeladene Bilder in sRGB zu konvertieren und sie vor dem Speichern entsprechend zu kennzeichnen?

(Ich würde gerne weiter recherchieren, wenn mir jemand die relevanten Code-Teile nennen könnte, die die Bildkonvertierung beim Hochladen steuern?)

codinghorror · 4. März 2016 um 02:53

Sure check with @zogstrip.

Patrick_David · 4. März 2016 um 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. März 2016 um 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. März 2016 um 22:43

Tested fix:

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

sam · 18. Juli 2018 um 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. Juli 2018 um 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. Juli 2018 um 15:10

Probably :comments is what we want there?

tgxworld · 25. Juli 2018 um 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. Juli 2018 um 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.

Thema		Antworten	Aufrufe
Image uploads - color profiles dropped Product Support uploads	4	687	23. September 2025
Stripping image metadata when composer media optimization is enabled? Product Support	5	685	15. Juli 2021
No strip image metadata works for PNG but not JPG Bug	9	1036	5. Mai 2022
Uploading image makes it black (not darker!) Bug	13	3816	3. April 2015
Site setting for "strip image metadata" Announcements	2	1787	25. Juli 2017

Eingebettetes Farbprofil im Bild

Verwandte Themen