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

Add test to reproduce multi-threading issue of computing extent and relationship #3528

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

roggiezhang-nv
Copy link
Contributor

@roggiezhang-nv roggiezhang-nv commented Feb 10, 2025

Description of Change(s)

We found a very strange behavior that ComputeExtent along with GetForwardedTargets in a multi-threading way would influence the behavior of getting material binding with API UsdRelationship::GetForwardedTargets. See the changes of this PR about the test to reproduce this. In this test, it's trying to traverse the whole stage with multi-threads. All operations are supposed to be read-only to the stage, and the test should be successful. However, it fails unexpectedly sometimes. Adding TfRegistryManager::GetInstance().SubscribeTo<UsdGeomBoundable>(); to the first line of main() will pass the test, which we still have no idea why.

You need to run the test with --repeat until-fail:1000 to increase the possibility of failure.

Fixes Issue(s)

#3529

Checklist

[x] I have created this PR based on the dev branch

[x] I have followed the coding conventions

[ ] I have added unit tests that exercise this functionality (Reference:
testing guidelines)

[ ] I have verified that all unit tests pass with the proposed changes

[x] I have submitted a signed Contributor License Agreement (Reference:
Contributor License Agreement instructions)

@roggiezhang-nv
Copy link
Contributor Author

@nvmkuruc for vis.

@nvmkuruc
Copy link
Collaborator

It was a little harder to reproduce on Linux, but I was able to reproduce with --repeat-until-fail 10000.

@jesschimein
Copy link
Collaborator

Filed as internal issue #USD-10665

(This is an automated message. See here for more information.)

@jesschimein
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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