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

"Filmic" tone mapping is not properly ported #2

Open
rhys-vdw opened this issue Dec 14, 2018 · 4 comments
Open

"Filmic" tone mapping is not properly ported #2

rhys-vdw opened this issue Dec 14, 2018 · 4 comments

Comments

@rhys-vdw
Copy link

rhys-vdw commented Dec 14, 2018

The color grading "mode" needs to be set to "High Definition Range" for this option to be available.

@AdamFrisby
Copy link
Owner

AdamFrisby commented Dec 14, 2018 via email

@rhys-vdw
Copy link
Author

rhys-vdw commented Dec 14, 2018

Ah, cool. I guess maybe if anything it could print a warning when it finds that invalid combo.

Really I feel like Unity devs have created and be maintaining this package.

@rhys-vdw
Copy link
Author

(It's super useful though, thanks!)

@AdamFrisby
Copy link
Owner

Yeah I'm trying to get a fix for this - I'm running into this myself. The issue is that tonemapping isn't available in LDR (it was before, -ish) --- and I can't port the YRGB curves in HDR (grrr) - I think we need to add a custom PostFX filter which does a YRGB adjustment -after- the colour grading, which allows us to use the curves again.

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

No branches or pull requests

2 participants