Skip to content

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

Merged
doodlum merged 4 commits into
community-shaders:devfrom
ThePagi:patch-4
Feb 16, 2025
Merged

fix(pbr): fix mlp flicker and wrong coat color, convert additional color maps like albedo#967
doodlum merged 4 commits into
community-shaders:devfrom
ThePagi:patch-4

Conversation

@ThePagi
Copy link
Copy Markdown
Contributor

@ThePagi ThePagi commented Feb 15, 2025

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

@doodlum
Copy link
Copy Markdown
Collaborator

doodlum commented Feb 16, 2025

we shouldn't flatten actually

@ThePagi
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator

doodlum commented Feb 16, 2025

ready for merge?

@doodlum doodlum merged commit de45037 into community-shaders:dev Feb 16, 2025
alandtse pushed a commit to alandtse/open-shaders that referenced this pull request Jul 20, 2025
…lor maps like albedo (community-shaders#967)

* fix(pbr): fix mlp flicker

* style: 🎨 apply clang-format changes

* Update Lighting.hlsl

* style: 🎨 apply clang-format changes

---------

Co-authored-by: ThePagi <ThePagi@users.noreply.github.com>
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