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

Time varying attributes #140

Closed
dedoardo opened this issue Apr 13, 2021 · 0 comments · Fixed by #148
Closed

Time varying attributes #140

dedoardo opened this issue Apr 13, 2021 · 0 comments · Fixed by #148
Assignees
Labels
in discussion Still requires a discussion question Further information is requested

Comments

@dedoardo
Copy link
Contributor

Time varying attributes - transforms, positions, normals or others we will need in the future - are sampled inconsistently and I think they should go behind the same interface similarly to HdCyclesSetTransform .

  • You make no assumptions about the samples/times are received
  • You approximate the input using a (configurable) number of samples which linearly interpolates them and formats them as Cycles like.

IMHO, this is something that should go inside Cycles and exposed through something like set_timevarying_attributes(times, samples).

I would say this has less priority than making the Sync process more parallel.

@dedoardo dedoardo added in discussion Still requires a discussion question Further information is requested labels Apr 13, 2021
@bareya bareya linked a pull request Apr 29, 2021 that will close this issue
@bareya bareya self-assigned this May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in discussion Still requires a discussion question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants