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.

Sign up for GitHub

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

Jump to bottom

Deadlock when calling AddTarget() with a relative path like <../.xtra_namespace> #76

Closed
blevin opened this issue Oct 6, 2016 · 3 comments
Closed

Deadlock when calling AddTarget() with a relative path like <../.xtra_namespace> #76

blevin opened this issue Oct 6, 2016 · 3 comments

Comments

@blevin
Copy link
Member

blevin commented Oct 6, 2016 •
edited
Loading

Daniel Berry reports a deadlock when trying to add a relative path in usd:

https://groups.google.com/d/msg/usd-interest/ZHVas9tPVoY/obsyxSYJGwAJ

This turns out to be a bug in Usd_InstanceCache::IsPathMasterOrInMaster(), which does not handle relative paths correctly at the moment.

The repro is easy:

from pxr import Usd, Sdf
s = Usd.Stage.CreateInMemory()
p = s.DefinePrim("/ShotObject01")
a = p.CreateAttribute("xtra_namespace", Sdf.ValueTypeNames.String)
a.Set("ShotObject01")

b = s.DefinePrim(p.GetPath().AppendPath("geo"))
r = b.CreateRelationship("xtra_namespace")
r.AddTarget("../.xtra_namespace")
@dbaz
Copy link

dbaz commented Oct 6, 2016

Cheers Brett!

@jtran56
Copy link

jtran56 commented Oct 7, 2016

Filed as internal issue #138430.

pixar-oss pushed a commit that referenced this issue Oct 18, 2016
@blevin @pixar-oss
Deadlock with AddTarget() of a relative path.
9abba3b 
Fixes issue #76.

(Internal change: 1662430)
@sunyab
Copy link
Contributor

sunyab commented Dec 10, 2016

Commit has been merged into release v0.7.2, closing.

@sunyab sunyab closed this as completed Dec 10, 2016
@alexwheezy alexwheezy mentioned this issue Jun 19, 2017
Closed
@asluk asluk mentioned this issue May 8, 2019
Closed
@Felix-Kit Felix-Kit mentioned this issue Feb 9, 2022
Closed
AdamFelt pushed a commit to autodesk-forks/USD that referenced this issue Apr 16, 2024
@PierreWang
Add automatic generated files and enable text rendering in USDView (P…
754b450 
…ixarAnimationStudios#76)

UsdView still show nothing for text, because the implementation is not ready.

(cherry picked from commit 5b8b03705b63a8cad9503516d2cbf8b1a8c9a64a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@blevin @dbaz @sunyab @jtran56