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

Changing topology only works when rendering the first frame of the USD file #564

Closed
kreppene opened this issue Oct 5, 2020 · 4 comments · Fixed by #676
Closed

Changing topology only works when rendering the first frame of the USD file #564

kreppene opened this issue Oct 5, 2020 · 4 comments · Fixed by #676
Assignees
Labels
bug Something isn't working procedural Related to the Arnold Procedural user Reported by a user

Comments

@kreppene
Copy link
Contributor

kreppene commented Oct 5, 2020

Describe the bug
When rendering a prim with changing topology, only the first frame of the usd file works,
on other frames i get messages like this
Error: [polymesh] p_liquidMilk_001/p_liquidMilk_001Shape: wrong number of vertex indices (found 189872, expected 200200) //

we should also make sure we use velocities when avalible

To Reproduce
Try rendering different frames of a usd file with changing topology,

Used Software Versions

  • Arnold: 6.0.4.0
  • USD: 0.20.05
  • Compiler: vs
  • OS: Windows 10
@kreppene kreppene added the bug Something isn't working label Oct 5, 2020
@compso compso added procedural Related to the Arnold Procedural user Reported by a user labels Nov 6, 2020
@bsilvaAX
Copy link

bsilvaAX commented Feb 5, 2021

Hello! To clarify, does this only apply to a monolithic USD file? i.e. if we were to split this into per-frame files, this error could be avoided?

@sebastienblor
Copy link
Collaborator

Hello! To clarify, does this only apply to a monolithic USD file? i.e. if we were to split this into per-frame files, this error could be avoided?

possibly yes, but then you'd be missing motion blur on these geometries. What we need to do is to read the velocities when they're present, and in this case we shouldn't try to look at how the topology changes.

Maybe we could check if the topology changes, and in this case we would only use static data. Not having motion blur might be better than not being able to render at all

@bsilvaAX
Copy link

bsilvaAX commented Feb 26, 2021

Ah, thanks! I see velocity motion blur is also on the to-do list.

I agree that it would be better to have the cache render w/o motion blur than not at all. But of course, supporting changing topology and velocity motion blur is vital for FX work (in particular).

@jhodgson
Copy link
Contributor

Example file with changing topology

test.usd_rop1.usda.zip

@sirpalee sirpalee changed the title Changing topology Changing topology only works when rendering the first frame of the USD file Apr 29, 2021
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 user Reported by a user
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants