Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Color management in settings seems to be ambiguous? #781

Closed
fieliapm opened this issue May 14, 2020 · 6 comments
Closed

Color management in settings seems to be ambiguous? #781

fieliapm opened this issue May 14, 2020 · 6 comments

Comments

@fieliapm
Copy link

System information:

  • Windows OS version: Windows 7, Winidows 10
  • ImageGlass version: 7.6.4.30
  • Monitor: Dell 2408WFP (gamut is as wide as AdobeRGB, host OS is Windows)
    Macbook Pro (Display P3 gamut, Windows OS in VM)

Questions

Hello, I am newbie to ImageGlass.
I came here to give ImageGlass a try because I could assign color profile in ImageGlass settings.
But the color profile is weird compare to other color management aware image viewers and image editors.
When I watched the photo (AdobeRGB) I take using ImageGlass, it is under saturated.
And after doing some test, I could conclude that
the custom color profile field in ImageGlass settings seems to be for output (ex. display), not for input,
and I cannot assign default icc for images without embedded icc.

However, ideal monitor does not exist, so every monitor would offer their own icm to tell software how to convert CIE color space to RGB to which monitor refers.
and ImageGlass list some predefined color profile
So I think custom color profile might be for input, but not for output, isn't it?

When doing color management, the correct flow should be:

original image (RGB, CMYK, YUV)
|
embedded icc, or specified icc (when embedded icc does not exist)
|
CIE color space
|
monitor icm
|
display RGB data
@d2phap
Copy link
Owner

d2phap commented May 29, 2020

Thanks @fieliapm for the details.

IG does not process the below steps:

CIE color space
|
monitor icm

@fieliapm
Copy link
Author

Don’t mention it. 😄
Hope ImageGlass will become one of the best open source image viewer so I can totally get rid of closed source ones on Windows.

@d2phap
Copy link
Owner

d2phap commented Jul 11, 2020

Hi @fieliapm

Can you try the latest build of ImageGlass Moon at https://imageglass.org/moon, I just upgraded Magick.NET library, it fixed CMYK rendering as mentioned in #779
May it also fix your problem?

@PaulCoddington
Copy link

Suggestion for Color Management

Modern wide-gamut monitors are switched frequently between different color spaces using a taskbar application. The taskbar application swaps the monitor ICC profile registered with Windows "on the fly" to match the monitor.

It would be good to have an option for image glass to automatically pick up the system registered profile for the monitor rather than have the user select a single profile from a list.

A lot of applications force the user to select a profile from a list, but this is very burdensome when there are so many applications to manually change when you want to flick between wide-gamut photo (Adobe RGB), video gamuts (DCI-P3, BT.709) and desktop/web (sRGB). The more applications that pick up the current ICC profile for the monitor automatically without fuss, the better for users.

@d2phap
Copy link
Owner

d2phap commented Sep 28, 2020

Hi @fieliapm
Could you help to try the latest build at https://imageglass.org/moon/
I suspect this is same as #779

@d2phap
Copy link
Owner

d2phap commented May 21, 2022

This issue is fixed in the latest build of ImageGlass Moon: https://imageglass.org/moon

  • Use current monitor profile is set as default option in Color management setting. If the monitor has no color profile, ImageGlass will use sRGB.
  • Moving ImageGlass window to another monitor does not auto-correct the color, you need to reload the image (Main menu > File > Reload image (Ctrl+R)). This is because reloading entire image file content is very costly.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants