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] add support for inputs:diffuse #15

Open
wants to merge 1 commit into
base: pr/hdEmbree-double-sided-mesh
Choose a base branch
from

Conversation

pmolodo
Copy link

@pmolodo pmolodo commented Jul 31, 2024

Description of Change(s)

hdEmbree multiples the inputs:diffuse attribute into it's lighting calculation, since currently it assumes a material that is 100% diffuse.


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-tf-debug-create-light branch from a7f9af6 to f8d4638 Compare August 1, 2024 18:46
@pmolodo pmolodo force-pushed the pr/hdEmbree-tf-debug-create-light branch from f8d4638 to 1dede61 Compare August 15, 2024 21:13
@pmolodo pmolodo force-pushed the pr/hdEmbree-tf-debug-create-light branch from 1dede61 to d33251e Compare September 23, 2024 18:11
@pmolodo pmolodo force-pushed the pr/hdEmbree-tf-debug-create-light branch from d33251e to 3ac23aa Compare October 21, 2024 17:06
@pmolodo pmolodo force-pushed the pr/hdEmbree-tf-debug-create-light branch from 3ac23aa to 05ff0ad Compare October 22, 2024 18:11
@pmolodo pmolodo changed the base branch from pr/hdEmbree-tf-debug-create-light to pr/hdEmbree-double-sided-mesh October 22, 2024 18:20
@pmolodo pmolodo force-pushed the pr/hdEmbree-double-sided-mesh branch from 65f44f3 to 2559f99 Compare October 23, 2024 17:29
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.

1 participant