{"action":"created","issue":{"url":"https://api.github.com/repos/LumaPictures/usd-qt/issues/9","repository_url":"https://api.github.com/repos/LumaPictures/usd-qt","labels_url":"https://api.github.com/repos/LumaPictures/usd-qt/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/LumaPictures/usd-qt/issues/9/comments","events_url":"https://api.github.com/repos/LumaPictures/usd-qt/issues/9/events","html_url":"https://github.com/LumaPictures/usd-qt/issues/9","id":356773301,"node_id":"MDU6SXNzdWUzNTY3NzMzMDE=","number":9,"title":"HierarchyStandardModel: invalid icon filepath?","user":{"login":"csaez","id":2292742,"node_id":"MDQ6VXNlcjIyOTI3NDI=","avatar_url":"https://avatars1.githubusercontent.com/u/2292742?v=4","gravatar_id":"","url":"https://api.github.com/users/csaez","html_url":"https://github.com/csaez","followers_url":"https://api.github.com/users/csaez/followers","following_url":"https://api.github.com/users/csaez/following{/other_user}","gists_url":"https://api.github.com/users/csaez/gists{/gist_id}","starred_url":"https://api.github.com/users/csaez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/csaez/subscriptions","organizations_url":"https://api.github.com/users/csaez/orgs","repos_url":"https://api.github.com/users/csaez/repos","events_url":"https://api.github.com/users/csaez/events{/privacy}","received_events_url":"https://api.github.com/users/csaez/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-09-04T11:46:28Z","updated_at":"2018-09-05T02:03:25Z","closed_at":null,"author_association":"NONE","body":"Hi,\r\nI'm not sure if this is a bug, but while trying to make the `HierarchyStandardModel` icons work today I realized that the running code is moved to `pxr.UsdQt` namespace after build, leaving the icons behind (`pxr.usdQt`) and making the relative paths to fail.\r\nhttps://github.com/LumaPictures/usd-qt/blob/35b01eaf71c5ebaf6262892ddc5584a5cae45184/pxr/usdQt/hierarchyModel.py#L279-L280\r\n\r\nIs there any plans to integrate the use of resources with the build system? Or even better, moving to QResource?\r\nIs this something we can help with?\r\n\r\nCheers!"},"comment":{"url":"https://api.github.com/repos/LumaPictures/usd-qt/issues/comments/418575131","html_url":"https://github.com/LumaPictures/usd-qt/issues/9#issuecomment-418575131","issue_url":"https://api.github.com/repos/LumaPictures/usd-qt/issues/9","id":418575131,"node_id":"MDEyOklzc3VlQ29tbWVudDQxODU3NTEzMQ==","user":{"login":"nrusch","id":730555,"node_id":"MDQ6VXNlcjczMDU1NQ==","avatar_url":"https://avatars3.githubusercontent.com/u/730555?v=4","gravatar_id":"","url":"https://api.github.com/users/nrusch","html_url":"https://github.com/nrusch","followers_url":"https://api.github.com/users/nrusch/followers","following_url":"https://api.github.com/users/nrusch/following{/other_user}","gists_url":"https://api.github.com/users/nrusch/gists{/gist_id}","starred_url":"https://api.github.com/users/nrusch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nrusch/subscriptions","organizations_url":"https://api.github.com/users/nrusch/orgs","repos_url":"https://api.github.com/users/nrusch/repos","events_url":"https://api.github.com/users/nrusch/events{/privacy}","received_events_url":"https://api.github.com/users/nrusch/received_events","type":"User","site_admin":false},"created_at":"2018-09-05T02:03:25Z","updated_at":"2018-09-05T02:03:25Z","author_association":"MEMBER","body":"Hi @csaez, thanks for the heads-up. We aren't using the icons internally, so I hadn't run into this one. It doesn't look like they're being installed at all by the build process, so that would definitely be one of the first things to fix.\r\n\r\nI also think it would be great to avoid hardcoding paths to resources into code, so I would be in favor of moving to `QResource`. If you're interested in tackling that, or have already done it internally and are able to submit it back as a PR, I think it make a welcome addition."}}
{"actor":{"display_login":"nrusch"}}