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

Experimental Raytrace now uses OkLCh as default and removes alternatives #423

Merged
merged 2 commits into from
Aug 3, 2024

Conversation

facelessuser
Copy link
Owner

Pre-configured oklch-raytrace and lch-raytrace are no longer provided. Users can easily recreate such pre-configured methods and we now document how.

CIELab when gamut mapped tightly, as the ray trace method does, gives very washed out yellows for extreme cases due to the geometry of the space. Oklab does much better in this area. CIELab will give more consistent results at general extremes, while Oklab breaks down near where the visible spectrum ends, especially in the blue region. We will go with the Oklab as more practical colors within the visible spectrum map fairly decently and people coming from CSS will likely prefer similar behavior. The option to use CIELAB, Luv, etc. are readily available for anyone that requires it.

Pre-configured oklch-raytrace and lch-raytrace are no longer provided.
Users can easily recreate such pre-configured methods and we now
document how.

CIELab when gamut mapped tightly, as the ray trace method does, gives
very washed out yellows for extreme cases due to the geometry of the
space. Oklab does much better in this area. CIELab will give more
consistent results at general extremes, while Oklab breaks down near
where the visible spectrum ends, especially in the blue region. We will
go with the Oklab as more practical colors within the visible spectrum
map fairly decently and people coming from CSS will likely prefer
similar behavior. The option to use CIELAB, Luv, etc. are readily
available for anyone that requires it.
@gir-bot gir-bot added S: needs-review Needs to be reviewed and/or approved. C: docs Related to documentation. C: source Related to source code. C: tests Related to testing. labels Aug 3, 2024
@gir-bot gir-bot added the C: infrastructure Related to project infrastructure. label Aug 3, 2024
@facelessuser
Copy link
Owner Author

@gir-bot lgtm

@gir-bot gir-bot added S: approved The pull request is ready to be merged. and removed S: needs-review Needs to be reviewed and/or approved. labels Aug 3, 2024
@facelessuser facelessuser merged commit e32f0b3 into main Aug 3, 2024
16 checks passed
@facelessuser facelessuser deleted the cleanup-raytrace branch August 3, 2024 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: docs Related to documentation. C: infrastructure Related to project infrastructure. C: source Related to source code. C: tests Related to testing. S: approved The pull request is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants