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] ensure we respect PXR_WORK_THREAD_LIMIT #16

Conversation

pmolodo
Copy link

@pmolodo pmolodo commented Jul 31, 2024

Description of Change(s)

Fixes an issue where pre-oneTBB, threads spawned from the non-root thread (such as the render thread) would not respect PXR_WORK_THREAD_LIMIT

Fixes Issue(s)

  • renderThread-spawned processes don't respect PXR_WORK_THREAD_LIMIT

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-inputs-diffuse branch from db06d9a to 5c78c8c Compare August 1, 2024 18:46
@pmolodo pmolodo force-pushed the pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT branch from 75af09e to 0425323 Compare August 1, 2024 18:46
@pmolodo pmolodo force-pushed the pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT branch from 0425323 to 820adf8 Compare August 15, 2024 21:13
@pmolodo pmolodo force-pushed the pr/hdEmbree-inputs-diffuse branch from 5c78c8c to a6e65e2 Compare August 15, 2024 21:13
@pmolodo pmolodo force-pushed the pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT branch from 820adf8 to b3d438c Compare September 23, 2024 18:11
@pmolodo pmolodo force-pushed the pr/hdEmbree-inputs-diffuse branch from a6e65e2 to 7dfd37b Compare September 23, 2024 18:11
@pmolodo pmolodo force-pushed the pr/hdEmbree-inputs-diffuse branch from 7dfd37b to e7c83dc Compare October 21, 2024 17:06
@pmolodo pmolodo closed this Oct 21, 2024
@pmolodo pmolodo force-pushed the pr/hdEmbree-respect-PXR_WORK_THREAD_LIMIT branch from b3d438c to fbdd452 Compare October 21, 2024 17:06
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