Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Error compiling against USD 21.05 #80

Open
DavidArayan opened this issue Jun 10, 2021 · 5 comments · May be fixed by #82
Open

Error compiling against USD 21.05 #80

DavidArayan opened this issue Jun 10, 2021 · 5 comments · May be fixed by #82

Comments

@DavidArayan
Copy link

Hello,

We are trying to compile this tool against USD version 21.05 and getting the following errors. It looks like some API changes have been made.

Screen Shot 2021-06-10 at 12 02 45 pm

We have attempted to build using both of these commands, leading to the same error

python3 ufgsrc/tools/ufginstall/ufginstall.py ufg ${USD_BUILD_PATH}

and

python ufgsrc/tools/ufginstall/ufginstall.py ufg ${USD_BUILD_PATH}

we have both python-dev and python3-dev also installed.

Any help is much appreciated, thank you!

@Maeroplane
Copy link

I get the exact same error with the attempt below

python {UFG_SRC}/tools/ufginstall/ufginstall.py {UFG_BUILD} {USD} --testdata

Any help is also much appreciated :)

@florihupf
Copy link

Looks like those calls need to be replaced with pbr_shader->GetPath().AppendProperty(kTokSurface) as needed. I think there are half a dozen in total.

@ElGuardianDelJardin
Copy link

ElGuardianDelJardin commented Jul 13, 2021

what said @florihupf worked for me, even tough i had to use . notation for most of the calls
pbr_shader.GetPath().AppendProperty(kTokSurface)

it built, but when I try to convert files, it gives me an error:
ERROR: USD: Failed connecting shading attribute </USDZ/Materials/Material.outputs:surface> to attribute surface on prim /USDZ/Materials/Material/pbr_shader. The given source information is not valid (ConnectToSource) [UFG_ERROR_USD]

@omerts
Copy link

omerts commented Jul 16, 2021

For every place using *.ConnectToSource(a, b) chain .ConnectableAPI() to param a.
For example, from:

in.ConnectToSource(tex, connect_tok);

To:

in.ConnectToSource(tex.ConnectableAPI(), connect_tok);

Creating a PR for this

EDIT:
PR Created you can also view the needed changes there

@omerts omerts linked a pull request Jul 16, 2021 that will close this issue
@DreamEmulator
Copy link

+1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants