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 correction now uses a 3D texture. #26101

Closed
wants to merge 1 commit into from

Conversation

QbieShay
Copy link
Contributor

Breaks games that use the existing color correction.

demo usage:
https://gfycat.com/mammothlivelyblacklab

Model from Desmond Man

@QbieShay QbieShay requested a review from reduz as a code owner February 20, 2019 21:25
@Calinou
Copy link
Member

Calinou commented Feb 20, 2019

Looks good, this should make color correction more powerful 😃

Would it be possible to implement this in a way that still makes GradientTexture usable for color correction? Having to use an image editor for simple color grading sounds quite cumbersome to tweak. Perhaps there could be a 3D GradientTexture resource in a future version?

…sting color correction.

Co-authored-by: CptPotato
@QbieShay
Copy link
Contributor Author

@CptPotato I implemented your change, but in order to co-author you I need your email :<

@CptPotato
Copy link
Contributor

@QbieShay You can leave the commit as is if you'd like. I'd prefer to keep my mail address private.

Though, make sure to test the change as I haven't done that. It should be really obvious with a neutral 2³ or 3³ texture.

@QbieShay
Copy link
Contributor Author

@CptPotato Will test this evening, thank you very much for the tip!

@fire
Copy link
Member

fire commented May 24, 2019

@QbieShay Did you test?

@akien-mga akien-mga modified the milestones: 3.2, 4.0 May 29, 2019
@QbieShay
Copy link
Contributor Author

QbieShay commented Jun 4, 2019

@fire I did! Works!

@fire
Copy link
Member

fire commented Jun 4, 2019

There was some difficulty converting a png lookup table to a 3d texture. I have to do 16 horizontal and 1 vertical. It would probably be better to support a .cube to 3d texture directly, but it works for me too!

@aaronfranke
Copy link
Member

@QbieShay Is this still desired? If so, it needs to be rebased on the latest master branch, and reworked to support Vulkan.

If not, abandoned pull requests will be closed in the future as announced here.

@fire
Copy link
Member

fire commented May 13, 2020

I still desire this. It required some engine investigation though.

@QbieShay
Copy link
Contributor Author

Hi @aaronfranke ! I haven't looked into the Vulkan renderer yet. I'm sad that this PR didn't make it into 3.x, but I expected it due to compat problems. I'll close this PR and maybe open another one for Vulkan in the future!

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

Successfully merging this pull request may close these issues.

6 participants