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

Register velocities, acclerations, and local constant-interpolation primvars on PIs #1734

Conversation

marktucker
Copy link
Contributor

Description of Change(s)

Register velocities, acclerations, and local constant-interpolation primvars
on point instancer primitives as hydra-accessible primvars. Otherwise
instancer-level settings applied as primvars can't be accessed by the
delegate, and constant-interpolation primvars can't be used to affect
prototypes marked as instanceable (where inheriting primvars doesn't
work).

This is a less ambitious version of #1731, which additoinally adds support for inherited primvars.

…rimvars

on point instancer primitives as hydra-accessible primvars. Otherwise
instancer-level settings applied as primvars can't be accessed by the
delegate, and constant-interpolation primvars can't be used to affect
prototypes marked as instanceable (where inheriting primvars doesn't
work).
@jilliene
Copy link

jilliene commented Jan 6, 2022

Filed as internal issue #USD-7112

pixar-oss pushed a commit that referenced this pull request Mar 8, 2022
- Fixed typo causing testUsdViewMaterialEdits to fail.

- Fixed failure tolerance on testUsdImagingGLDomeLight_multi
  to avoid conflicts with perceptual diff as done in other
  perceptual diff tests. On our internal test machine,
  there are multiple pixel differences but no perceptual
  differences.

- Fixed missing install of Embree-related test assets
  that caused test failures.

- Changed several Embree test settings, primarily to use
  perceptual diffs instead of pixel diffs.

- Disabled testUsdviewLegacyImpl. This test is failing on
  an internal test machine and needs to be investigated.

- Disabled testusdview/usdImagingGL tests on Windows due
  to intermittent hangs that need to be investigated

(Internal change: 2217839)
@tcauchois
Copy link
Contributor

For both constant primvars (here) and inherited primvars (in the other PR), you're changing the hydra schema in a way where we'll need to fix up our backends to now pull constant instancer data, which makes it harder to land.

Is it possible to get just velocities and accelerations by themselves?

@marktucker
Copy link
Contributor Author

I can certainly split outhe vel/accel parts of this PR into a separate reduced version of this PR. I'm not sure I see the downside to exposing the constant interp primvars though. If a given render delegate doesn't want to look at these values, it doesn't have to, and its behavior will continue to be the same as if hydra wasn't reporting these primvars.

Anyway, I'll create the PR once the 22.05 dust settles a bit. I'll be busy upgrading for the next little while.

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

Successfully merging this pull request may close these issues.

3 participants