ملف الألوان المضمن في الصورة

ساهم خلل
1

لا أعرف إن كان هذا قد ظهر في مكان آخر (فكلمات البحث، ملف تعريف الألوان المضمن، تعيد نتائج غير مرتبطة.. :)).

أدير منتدى للبرمجيات الحرة في مجال التصوير الفوتوغرافي على discuss.pixls.us. وكما تتخيل، لدينا مستخدمون مهتمون جدًا بكيفية عرض صورهم.

من الأمور التي واجهناها مؤخرًا كانت مشاكل في موضوع حول إدارة الألوان:

لاحظ أحد المصورين أن وظيفة جلب الصور المرتبطة وتخزينها محليًا تبدو وكأنها تزيل معلومات الألوان من الصور. ورغم أن هذا قد يكون مقبولًا لتوفير المساحة في معظم المنتديات، إلا أنه في منتدى مخصص للتصوير الفوتوغرافي، فإن هذا الأمر مؤلم بعض الشيء. :frowning:

وللتوضيح، تحتوي هذه الصورة على ملف تعريف ألوان مضمن ومُدار بشكل خاطئ. يجب أن تظهر كما يلي (Rot = أحمر، Grün = أخضر، Blau = أزرق):

farbkreis
farbkreis800×800 82.9 KB

يمكن العثور على الصورة المصدر التي أدرجتها في المنشور أعلاه هنا:

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

وفيما يلي نفس الصورة، مُحمَّلة مباشرةً إلى المنتدى:

farbkreis.jpg800×800 176 KB

هل تتم معالجة الصور باستخدام برنامج ImageMagick؟ وإذا كان الأمر كذلك، هل هناك أي تفكير في إضافة خيار لتحويل الصور المرفوعة إلى مساحة ألوان sRGB ووضع علامة عليها كـ sRGB قبل الحفظ؟

(سأكون سعيدًا بالبحث بشكل أعمق إذا أمكن لأحد أن يشير إلى الأجزاء ذات الصلة من الكود التي تتعامل مع تحويل الصور عند رفعها؟)

5 إعجابات
2

Sure check with @zogstrip.

3

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

4

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

3 إعجابات
5

Tested fix:

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

3 إعجابات
6

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?

3 إعجابات
8

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 )
3 إعجابات
9

Probably :comments is what we want there?

إعجاب واحد (1)
10

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.

إعجابَين (2)
11

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.

5 إعجابات