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

TexCoord2 support for [DrawMesh] & [SetMaterial] #511

Draft
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

newemka
Copy link
Contributor

@newemka newemka commented Feb 1, 2025

Since I've updated the PBR struct for [DisplaceMeshVAT], a mesh can have a second set of texture coordinates.

TexCoord2-v01.mp4

Todo :

  • support for shared Blend-function.hlsl
  • TexCoord switches for the different textures
  • trying to not over bloat [SetMaterial]

@pixtur
Copy link
Collaborator

pixtur commented Feb 12, 2025

Hi @newemka,

I would have thought that many more than 14 files would require to be updated for that.
Shouldn't this affect all mesh operators?

@newemka
Copy link
Contributor Author

newemka commented Feb 12, 2025

You're right, since the PBR struct update , all mesh operators have their stride updated as well. (#479)
Right now the state of this PR is just what I needed for a project I'm working on. It allows me to make some experiments and see how to implement the use of the TexCoord2 for more operators.
I will update more mesh opertors soon.

@pixtur
Copy link
Collaborator

pixtur commented Feb 12, 2025

Okay. No worries. But it would probably be good to pull this over to main

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