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

fix(pbr): fix mlp flicker and wrong coat color, convert additional color maps like albedo #967

Merged
merged 4 commits into from
Feb 16, 2025

Conversation

ThePagi
Copy link
Contributor

@ThePagi ThePagi commented Feb 15, 2025

Adds a check to avoid wrongly converting MLP view dir on armor (smh)

@doodlum
Copy link
Owner

doodlum commented Feb 16, 2025

we shouldn't flatten actually

@ThePagi
Copy link
Contributor Author

ThePagi commented Feb 16, 2025

Copied the code with flatten from elsewhere, ig in another PR remove all flatten from related code.

I'll add more fixes to this pr cos I didn't switch branches 😐

@ThePagi ThePagi changed the title fix(pbr): fix mlp flicker fix(pbr): fix mlp flicker and wrong coat color, convert additional color maps like albedo Feb 16, 2025
@ThePagi
Copy link
Contributor Author

ThePagi commented Feb 16, 2025

Changes:
PBR MLP index of refraction affected by coat strength -> no wrong angle when no coat
PBR SSS, fuzz, glow, MLP colors converted with Color::Diffuse for 'correct' color space (glow was too dim etc.)

@doodlum
Copy link
Owner

doodlum commented Feb 16, 2025

ready for merge?

@doodlum doodlum merged commit de45037 into doodlum:dev Feb 16, 2025
2 checks passed
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

Successfully merging this pull request may close these issues.

2 participants