drwave / usd-sketchup

Source & macOS binary of USD exporter for SketchUp
Other
63 stars 5 forks source link

unsupported path names #9

Closed wave-electron closed 6 years ago

wave-electron commented 6 years ago

using this SketchUp model from 3Dwarehouse

clock

seems to display ok in xcode beta6, but crashes in IOS12 beta 12

clock

Does not have any errors when exporting to usdz from SketchUp. However, if I export as usda from SketchUp and use the xcrun usdz_converter to convert to usdz, I get the following complaint about path names.

iMac:COLLADA2GLTF ashleyjoyce$ xcrun usdz_converter clock.usda clock.usdz
usd_converter converting file: clock.usda
converted usdz file: clock.usdz
Coding Error: in _ValidateEditPrim at line 1157 of /Library/Caches/com.apple.xbs/Sources/ModelKit/ModelKit-1.94.23/ModelKit/extern/usd/pxr/usd/usd/stage.cpp -- Cannot create property spec at path </__Master_2/Materials/Material/Texture>; authoring to an instancing master is not allowed.
Runtime Error: in _SetValueImpl at line 1494 of /Library/Caches/com.apple.xbs/Sources/ModelKit/ModelKit-1.94.23/ModelKit/extern/usd/pxr/usd/usd/stage.cpp -- Cannot set attribute value.  Failed to create attribute spec </__Master_2/Materials/Material/Texture.inputs:file> in layer @/opt/COLLADA2GLTF/clock_$$usd_converter$$_.usdc@
wave-electron commented 6 years ago

Actually after opening up the accompanying file clock.components.usda. It seems like it has two instances of the clock displaying... so some issues there.

drwave commented 6 years ago

Hmm, not sure what you’re seeing. You should be opening up the main file, which will sublayer in the .geom.usda file, which will then reference the .components.usda file.

The .usdz and the .usda files load up fine for me.

Sounds like you should file a radar against usdz_converter...

On Sep 4, 2018, at 4:53 PM, Ashley Joyce notifications@github.com wrote:

Actually after opening up the accompanying file clock.components.usda. It seems like it has two instances of the clock displaying... so some issues there.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/drwave/usd-sketchup/issues/9#issuecomment-418554243, or mute the thread https://github.com/notifications/unsubscribe-auth/ABmgOmz-NrU3LdZ5GPYA5JP7l0wTuEdxks5uXxJzgaJpZM4WZ729.

drwave commented 6 years ago

I’m definitely able to repro what you’re seeing with usdz_converter, but it looks to be a bug with it, not the file I generated.

File a radar, give me the number, and I’ll follow up with the model IO team.

On Sep 4, 2018, at 5:25 PM, Michael B. Johnson drwave@me.com wrote:

Hmm, not sure what you’re seeing. You should be opening up the main file, which will sublayer in the .geom.usda file, which will then reference the .components.usda file.

The .usdz and the .usda files load up fine for me.

Sounds like you should file a radar against usdz_converter...

On Sep 4, 2018, at 4:53 PM, Ashley Joyce <notifications@github.com mailto:notifications@github.com> wrote:

Actually after opening up the accompanying file clock.components.usda. It seems like it has two instances of the clock displaying... so some issues there.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/drwave/usd-sketchup/issues/9#issuecomment-418554243, or mute the thread https://github.com/notifications/unsubscribe-auth/ABmgOmz-NrU3LdZ5GPYA5JP7l0wTuEdxks5uXxJzgaJpZM4WZ729.

wave-electron commented 6 years ago

ok. will do