Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AiSceneLoad does not load array type user parameter from usd #1315

Closed
cpichard opened this issue Oct 4, 2022 · 1 comment · Fixed by #1323 or #1340
Closed

AiSceneLoad does not load array type user parameter from usd #1315

cpichard opened this issue Oct 4, 2022 · 1 comment · Fixed by #1323 or #1340
Assignees
Labels
bug Something isn't working procedural Related to the Arnold Procedural

Comments

@cpichard
Copy link
Collaborator

cpichard commented Oct 4, 2022

Describe the bug
The attached archive has the test code.
The test code does the following:
create a node
declare a user array parameter on the node
save the node to .usd
load the node from .usd
verify if the node has the user array parameter set
The test fails with the following error:
WARNING | myimager: could not read ARRAY parameter "my_array"
[ERROR] 'my_array' is defined null
Changing the format to .ass works fine.
Writing to usda and looking at the file, I can tell that the array is written correctly, so it's likely an issue with the loader:
float[] primvars:my_array = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15] (
elementSize = 16
interpolation = "constant"
)

UsdUserArrayTest.cpp.zip

@cpichard cpichard added the bug Something isn't working label Oct 4, 2022
@sebastienblor sebastienblor added the procedural Related to the Arnold Procedural label Oct 7, 2022
@cpichard cpichard moved this to In Progress in Arnold USD Oct 14, 2022
@cpichard cpichard self-assigned this Oct 14, 2022
@cpichard cpichard moved this from In Progress to In review in Arnold USD Oct 25, 2022
Repository owner moved this from In review to Done in Arnold USD Oct 28, 2022
@sebastienblor sebastienblor reopened this Oct 29, 2022
Repository owner moved this from Done to Todo in Arnold USD Oct 29, 2022
@sebastienblor
Copy link
Collaborator

This change broke a high amount of tests, so it's been reverted. Re-opening the ticket

@cpichard cpichard moved this from Todo to In review in Arnold USD Nov 1, 2022
Repository owner moved this from In review to Done in Arnold USD Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working procedural Related to the Arnold Procedural
Projects
Status: Done
2 participants