Skip to content

USDZExporter: Fix Object indentation level#31126

Merged
Mugen87 merged 1 commit intomrdoob:devfrom
mkeblx:usdz-exp-fmt1
May 18, 2025
Merged

USDZExporter: Fix Object indentation level#31126
Mugen87 merged 1 commit intomrdoob:devfrom
mkeblx:usdz-exp-fmt1

Conversation

@mkeblx
Copy link
Contributor

@mkeblx mkeblx commented May 17, 2025

Description

Add 3 tabs for Object_x's as they should always be placed at 3 levels in with the current exporter fixed output structure.

Before:

def Xform "Root"
{
	def Scope "Scenes" (
		kind = "sceneLibrary"
	)
	{
		def Xform "Scene" (
			customData = {
				bool preliminary_collidesWithEnvironment = 0
				string sceneName = "Scene"
			}
			sceneName = "Scene"
		)
		{
def Xform "Object_15" (
	prepend references = @./geometries/Geometry_3.usda@</Geometry>
	prepend apiSchemas = ["MaterialBindingAPI"]
)
{
	matrix4d xformOp:transform = ( (1, 0, 0, 0), (0, 1, 0, 0), (0, 0, 1, 0), (0, 0, 0, 1) )
	uniform token[] xformOpOrder = ["xformOp:transform"]

	rel material:binding = </Materials/Material_6>
}
...

After:

def Xform "Root"
{
	def Scope "Scenes" (
		kind = "sceneLibrary"
	)
	{
		def Xform "Scene" (
			customData = {
				bool preliminary_collidesWithEnvironment = 0
				string sceneName = "Scene"
			}
			sceneName = "Scene"
		)
		{
			def Xform "Object_15" (
				prepend references = @./geometries/Geometry_3.usda@</Geometry>
				prepend apiSchemas = ["MaterialBindingAPI"]
			)
			{
				matrix4d xformOp:transform = ( (1, 0, 0, 0), (0, 1, 0, 0), (0, 0, 1, 0), (0, 0, 0, 1) )
				uniform token[] xformOpOrder = ["xformOp:transform"]

				rel material:binding = </Materials/Material_6>
			}
...

@Mugen87 Mugen87 added this to the r177 milestone May 18, 2025
@Mugen87 Mugen87 merged commit b654763 into mrdoob:dev May 18, 2025
11 checks passed
RuthySheffi pushed a commit to RuthySheffi/three.js that referenced this pull request Jun 5, 2025
RuthySheffi pushed a commit to RuthySheffi/three.js that referenced this pull request Jun 5, 2025
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