-
Notifications
You must be signed in to change notification settings - Fork 204
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
Using correct ICC profile while exporting Webp Images #410
Conversation
closes #384 |
I'm sorry @prashik-s but the test seems to be failing |
This reverts commit 5c0bc15.
Sorry for the hassle. I had to revert the merge |
I'm not sure what's wrong but the test didn't even execute. It failed even before this step |
Can you resubmit the PR? You can see the failed test in the main branch commit, not on your PR
|
you can see test run #560: `vips cac image_test.go:75:
--- FAIL: TestImageInbuildIcc_WebP (0.38s) |
Hey, I ran into a similar issue with ICC profiles. When converting a P3 Display image using the vips CLI command func convertToWebP(image *vips.ImageRef, quality int) (imageBytes []byte, err error) {
options := vips.NewWebpExportParams()
options.ReductionEffort = 4
options.Quality = quality
imageBytes, _, err = image.ExportWebp(options)
if err != nil {
return nil, fmt.Errorf("error exporting to webp: %w", err)
}
return imageBytes, nil
} |
…byttow#410)" (davidbyttow#412) This reverts commit 48c3867.
…byttow#410)" (davidbyttow#412) This reverts commit 48c3867.
…byttow#410)" (davidbyttow#412) This reverts commit 48c3867.
No description provided.