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

Improve the ray trace algorithm to yield better results in all spaces #435

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

facelessuser
Copy link
Owner

@facelessuser facelessuser commented Sep 11, 2024

The ray trace approach was designed focusing mainly on Oklab/OkLCh. Further evaluation shows that some spaces twist a little different through the RGB spaces and can yield results not as good, CIELCh being an example.

Adjust the algorithm so that if a point is found along the reduction path below the gamut surface, that point becomes the new anchor.

This generally tightness the range in a few passes and prevent some spaces from having larger than expected hue or lightness deviations.

@gir-bot gir-bot added S: needs-review Needs to be reviewed and/or approved. C: docs Related to documentation. C: infrastructure Related to project infrastructure. C: source Related to source code. C: tests Related to testing. labels Sep 11, 2024
The ray trace approach was designed focusing mainly on Oklab/OkLCh.
Further evaluation shows that some spaces twist a little different
through the RGB spaces and can yield results not as good, CIELCh
being an example.

Adjust the algorithm so that if a point is found along the reduction
path below the gamut surface, that point becomes the new anchor.

This generally tightness the range in a few passes and prevent some
spaces from having larger than expected hue deviations.
@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 Sep 12, 2024
@facelessuser facelessuser merged commit 7f7640f into main Sep 12, 2024
18 checks passed
@facelessuser facelessuser deleted the enhance/raytrace-improve branch September 12, 2024 03:41
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