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

[hdEmbree] Initial UsdLux reference implementation #4

Open
wants to merge 1 commit into
base: pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT
Choose a base branch
from

Conversation

pmolodo
Copy link

@pmolodo pmolodo commented Jul 31, 2024

Description of Change(s)

This modifies the hdEmbree example plugin to do direct lighting so as to provide a reference implementation of the expected behaviour for UsdLux.

If no lights are present in the stage, the old ambient occlusion path will be used.


NOTE: This chain of PRs was made separate from the PR documenting expected behavior:


  • I have verified that all unit tests pass with the proposed changes
  • I have submitted a signed Contributor License Agreement

This modifies the hdEmbree example plugin to do direct lighting so as to
provide a reference implementation of the expected behaviour for UsdLux.

If no lights are present in the stage, the old ambient occlusion path
will be used.
@pmolodo pmolodo force-pushed the pr/hdEmbree-usd-lux-ref-implementation branch from 9088c1b to 2c9ff46 Compare October 21, 2024 17:06
@pmolodo pmolodo changed the base branch from pr/hdEmbree-usd-lux-prep to pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT October 21, 2024 17:43
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.

2 participants