Yellow-Dog-Man / Resonite-Issues

Issue repository for Resonite.
https://resonite.com
140 stars 2 forks source link

PBS_MultiUV_Metallic and PBS_MultiUV_Specular do not respect Culling and AlphaHandling #825

Open Toxic-Cookie opened 11 months ago

Toxic-Cookie commented 11 months ago

Describe the bug?

When using a PBS_MultiUV_Metallic or PBS_MultiUV_Specular, the Culling field is completely ignored and the AlphaHandling field when specifically set to AlphaBlend, is also completely ignored.

To Reproduce

Create a PBS_MultiUV_Metallic or PBS_MultiUV_Specular material and then attempt to change the Culling field and observe nothing changing. Also set the AlphaHandling field to AlphaBlend and observe nothing changing when the alpha color is adjusted.

Expected behavior

I would expect the states of the Culling and AlphaHandling fields of PBS_MultiUV_Metallic and PBS_MultiUV_Specular to be respected.

Screenshots

20231130015546_1

Resonite Version Number

2023.11.28.1365

What Platforms does this occur on?

Windows

What headset if any do you use?

Desktop, Valve Index

Log Files

DESKTOP-RSFAV05 - 2023.11.28.1365 - 2023-11-30 04_09_23.log Player.log

Additional Context

No response

Reporters

Resonite: Toxic_Cookie Discord: toxic_cookie

shiftyscales commented 8 months ago

Assigning @Geenz to look into this at some point.

Geenz commented 7 months ago

Culling and alpha handling is not supported on these materials. This is more an effect of these materials sharing the same basic parameters as all other PBS materials. This is expected behavior. There's a lot more consideration we need to put into supporting this with the eventual PBS consolidation.

Toxic-Cookie commented 7 months ago

As far as I can tell, these materials are literally the exact same thing as their standard counterparts with the only difference being more UVs are supported. It seems like a strange and arbitrary decision to not have feature parody for certain properties like this.

Geenz commented 7 months ago

That may seem like it to you, but they are infact implemented differently from the standard PBS materials. This is an artifact of how many of those materials were implemented initially - there's several of those materials where culling and alpha blend modes simply don't work due to this.

The real solution here is to unify these things so all culling and blending modes work across all PBS material types. I'll stand up an issue reflecting this.

Geenz commented 7 months ago

I'm gonna reopen this one just for tracking in the associated milestone.