USD export: implement MAPPING node (texture scale, translation and rotation) #106684

New Issue

Can you add steps to reproduce this (a .blend file would also help here)?
Can this be observed just from within blender (roundtrip blender > export USD > import USD)?

Sorry this has been lying around for so long. Can you add steps to reproduce this (a .blend file would also help here)? Can this be observed just from within blender (roundtrip blender > export USD > import USD)?

Philipp Oeser added

Status

Needs Information from User

and removed

Status

Needs Triage

labels 2023-09-20 12:13:55 +02:00

Tim Knip commented

2023-09-21 13:24:11 +02:00

@lichtwerk No problem ;-) Here's a simple blend file to reproduce the issue.

The mapping node is ignored on export. And yes, it can be observed with an export > import roundtrip.

Steps to reproduce:

export to usd(z)
import the usd(z)

System info:

windows 11 pro
blender 3.6.0 - c7fc78b81e

@lichtwerk No problem ;-) Here's a simple blend file to reproduce the issue. The mapping node is ignored on export. And yes, it can be observed with an export > import roundtrip. Steps to reproduce: - export to usd(z) - import the usd(z) System info: - windows 11 pro - blender 3.6.0 - c7fc78b81ecb

USD-mapping-issue.usdz

284 KiB

after-import.png

356 KiB

USD-mapping-issue.blend

885 KiB

before-export.png

374 KiB

Philipp Oeser commented

2023-09-21 14:18:03 +02:00

Thx getting back!

So yeah, unfortunately the support for the mapping node is not implemented in the USD Preview Surface material export yet.

Not all nodes are supported; currently only Diffuse, Principle, Image Textures, and UVMap nodes are support.

See https://docs.blender.org/manual/en/4.0/files/import_export/usd.html and c85c52f2ce

I think we would be looking at implementing UsdTransform2d, looks like blenders mapping node does approximately the same.
Atm. all the code does is looking for a UV input upstream of the image texture node, see create_uvmap_shader in create_usd_preview_surface_material, it probably wouldnt be too hard to search for a mapping node as well.

That all being said, this is not considered a bug

Thanks for the report, but the issue reported here is a request for modified/improved behavior and not a bug in current behavior. Closing as this bug tracker is only for bugs and errors.

For user requests and feedback, please use other channels: https://wiki.blender.org/wiki/Communication/Contact#User_Feedback_and_Requests

For more information on why this isn't considered a bug, visit: https://wiki.blender.org/wiki/Reference/Not_a_bug

Afraid we have to close this (unless we want to keep this as a TODO? -- feel free to reopen in such case @dr.sybren)

Thx getting back! So yeah, unfortunately the support for the mapping node is not implemented in the USD Preview Surface material export yet. >Not all nodes are supported; currently only Diffuse, Principle, Image Textures, and UVMap nodes are support. See https://docs.blender.org/manual/en/4.0/files/import_export/usd.html and c85c52f2ce47 I think we would be looking at implementing `UsdTransform2d`, looks like blenders mapping node does approximately the same. Atm. all the code does is looking for a UV input upstream of the image texture node, see `create_uvmap_shader` in `create_usd_preview_surface_material`, it probably wouldnt be too hard to search for a mapping node as well. That all being said, this is not considered a bug ``` Thanks for the report, but the issue reported here is a request for modified/improved behavior and not a bug in current behavior. Closing as this bug tracker is only for bugs and errors. For user requests and feedback, please use other channels: https://wiki.blender.org/wiki/Communication/Contact#User_Feedback_and_Requests For more information on why this isn't considered a bug, visit: https://wiki.blender.org/wiki/Reference/Not_a_bug ``` Afraid we have to close this (unless we want to keep this as a TODO? -- feel free to reopen in such case @dr.sybren)

Philipp Oeser closed this issue

2023-09-21 14:18:06 +02:00

Blender Bot added

Status

Archived

and removed

Status

Needs Information from User

labels 2023-09-21 14:19:03 +02:00

Sign in to join this conversation.