Proposal: rename texture.texCoordTransformMatrix to texture.transform about pex-renderer HOT 9 CLOSED

Which reminds me of #310 Consider renaming maps to textures which would cause

uBaseColorMap
uBaseColorMapTexCoordTransform

uBaseColorTexture
uBaseColorTextureTransform //yay!

from pex-renderer.

So new components will look like that:

// basic
const material = {
  baseColorTexture: texture
};

// or with transform
const material = {
  baseColorTexture: {
     texture,
     transform: textureTransform
  }
};

Fine by me. Renaming in pex-shaders is needed as well yes, so that's a big breaking change where we need to be careful with VSCode search & replace.

from pex-renderer.

Well it's actually not that simple as transform in KHR_texture_transform is an object with offset, rotation and scale and not a matrix so it could be confusing.

So it would have to baseColorTexture.transformMatrix (still shorter) and uBaseColorTextureTransformMatrix

from pex-renderer.

At the moment, materials texture.texCoordTransformMatrix is not set directly. Instead, the more user friendly texture.offset/rotation/scale is used and the matrix only computed on set(). So you can either provide:

• { texture, offset?, rotation?, scale? }
• { texture, texCoordTransformMatrix }

The three questions:

• is renaming texCoordTransformMatrix to transformMatrix okay
• can we still allow texture.offset/rotation/scale and compute texture.transformMatrix in material system (render-pipeline?)
• can we animate update these properties at runtime, if so, how do you tell texture.transformMatrix to be recomputed?

from pex-renderer.

Instead, the more user friendly texture.offset/rotation/scale is used

Is it? :D I thought all we have is texCoordTransformMatrix

is renaming texCoordTransformMatrix to transformMatrix okay

Yes

can we still allow texture.offset/rotation/scale and compute texture.transformMatrix in material system (render-pipeline?)

How to name and pack given uniforms is up to renderer. Are there any other use cases for Material System? As of now the most appropriate place is standard material.

can we animate update these properties at runtime, if so, how do you tell texture.transformMatrix to be recomputed?

Are we thinking arcade game scrolling textures here? :)

from pex-renderer.

Instead, the more user friendly texture.offset/rotation/scale is used

Is it? :D I thought all we have is texCoordTransformMatrix

Yep, here we compute the matrix:

can we still allow texture.offset/rotation/scale and compute texture.transformMatrix in material system (render-pipeline?)

How to name and pack given uniforms is up to renderer. Are there any other use cases for Material System? As of now the most appropriate place is standard material.

No but maybe in a shared space if basic renderer or any other wants to use textures and texture transform?

can we animate update these properties at runtime, if so, how do you tell texture.transformMatrix to be recomputed?

Are we thinking arcade game scrolling textures here? :)

Thinking texture spritesheet or something ;)

from pex-renderer.

Oh that's leftover code that has not been ported yet. The thing with material system is that renderers are building materials atm. And texture matrix update system would have to know all possible texture names or sniff for objects with texture property in material components, not optimal.

My plan for standard material was to support all possible 2d/3d mesh like scenarios that would involve textures and where texture transforms make sense. Other renderers like particles/lines/trails would less. Although i see usecase for spritesheets in those. Will it be full matrix with translate rotate scale though? Or rather offset, size as vec4? Just don't want to force generalization that is not there.

from pex-renderer.

Update.

Given that we udate uBaseColorMap to uBaseColorTexture we would end up with uBaseColorTextureTexCoordTransform so maybe we should rename this too to uBaseColorTextureTransformMatrix and following material prop:

const material = {
  baseColorMap: {
     texture: texture,
     transformMatrix: mat3
  }
}

from pex-renderer.

Suggested naming:

const material = {
  baseColorTexture: {
     texture: texture,
     matrix: mat3,
     texCoord: int,
  }
}

uniform mat3 uBaseColorTextureMatrix;

Flags:
USE_BASE_COLOR_TEXTURE
USE_BASE_COLOR_TEXTURE_MATRIX
BASE_COLOR_TEXTURE_TEX_COORD

from pex-renderer.