Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Fill color is too light #1541

Closed
kkaefer opened this issue May 13, 2015 · 8 comments
Closed

Fill color is too light #1541

kkaefer opened this issue May 13, 2015 · 8 comments
Labels

Comments

@kkaefer
Copy link
Member

kkaefer commented May 13, 2015

Our expected image is black rgb(0, 0, 0):

image

However, we're getting rgb(1, 1, 1):

image

(ignore the border around the tile)

@kkaefer
Copy link
Member Author

kkaefer commented May 13, 2015

Seeing this on OS X 10.10.3 with a nVidia GeForce GT 650M.

@kkaefer
Copy link
Member Author

kkaefer commented May 13, 2015

This may be due to sRGB bits:

image

image

@tmpsantos
Copy link
Contributor

I get the same here on Ubuntu 15.04 Vivid with an ATI Radeon HD 6850.

@tmpsantos
Copy link
Contributor

I get the same here on Ubuntu 15.04 Vivid with an ATI Radeon HD 6850.

Nah, my bad. On fill-color/default here only the borders are different.

@kkaefer
Copy link
Member Author

kkaefer commented May 13, 2015

Confirmed that this is the cause of images appearing too dark or bright compared to the reference image; the actual image data is not different. We set the rendering intent to default, but it looks like Core Graphics is still outputting perceptual, or ICC color profiles.

@jfirebaugh
Copy link
Contributor

@kkaefer Can you test this and #1542 against https://github.com/mapbox/mapbox-gl-test-suite/compare/update -- I updated the tests there using current gl-js and can no longer replicate either. Maybe it is due to a library on my machine being upgraded since the expected results were last generated.

@kkaefer
Copy link
Member Author

kkaefer commented May 20, 2015

@jfirebaugh
Copy link
Contributor

@kkaefer Can you test this again? I think it was fixed by either always running headless tests in node, or by #3083.

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

No branches or pull requests

3 participants