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

@pmolodo pmolodo force-pushed the pr/hdEmbree-usd-lux-ref-implementation branch from 34d9a31 to a6a9136 Compare August 1, 2024 18:46
@pmolodo pmolodo force-pushed the pr/hdEmbree-usd-lux-ref-implementation branch from a6a9136 to 9088c1b Compare September 23, 2024 18:11
@pmolodo pmolodo force-pushed the pr/hdEmbree-usd-lux-prep branch 2 times, most recently from dabdeb5 to 5d519c7 Compare September 23, 2024 18:23
@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
@pmolodo pmolodo force-pushed the pr/hdEmbree-usd-lux-ref-implementation branch from 2c9ff46 to c0aebc7 Compare October 22, 2024 18:11
@pmolodo pmolodo force-pushed the pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT branch from fbdd452 to 0a241c1 Compare October 22, 2024 18:11
@pmolodo pmolodo force-pushed the pr/hdEmbree-usd-lux-ref-implementation branch 2 times, most recently from be07932 to 76c2135 Compare October 23, 2024 18:20
@pmolodo pmolodo force-pushed the pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT branch from 0a241c1 to 89e974f Compare October 23, 2024 18:20
@pmolodo pmolodo force-pushed the pr/hdEmbree-usd-lux-ref-implementation branch from 76c2135 to db8377c Compare January 28, 2025 22:27
@pmolodo pmolodo force-pushed the pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT branch from 89e974f to 73cdf8a Compare January 28, 2025 22:27
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 db8377c to 3a6afbf Compare February 5, 2025 17:28
@pmolodo pmolodo force-pushed the pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT branch from 73cdf8a to d6a0b09 Compare February 5, 2025 17:28
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