{"action":"created","issue":{"url":"https://api.github.com/repos/mottosso/Qt.py/issues/237","repository_url":"https://api.github.com/repos/mottosso/Qt.py","labels_url":"https://api.github.com/repos/mottosso/Qt.py/issues/237/labels{/name}","comments_url":"https://api.github.com/repos/mottosso/Qt.py/issues/237/comments","events_url":"https://api.github.com/repos/mottosso/Qt.py/issues/237/events","html_url":"https://github.com/mottosso/Qt.py/issues/237","id":252125702,"number":237,"title":"loadUiType usage","user":{"login":"Ahuge","id":704821,"avatar_url":"https://avatars0.githubusercontent.com/u/704821?v=4","gravatar_id":"","url":"https://api.github.com/users/Ahuge","html_url":"https://github.com/Ahuge","followers_url":"https://api.github.com/users/Ahuge/followers","following_url":"https://api.github.com/users/Ahuge/following{/other_user}","gists_url":"https://api.github.com/users/Ahuge/gists{/gist_id}","starred_url":"https://api.github.com/users/Ahuge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ahuge/subscriptions","organizations_url":"https://api.github.com/users/Ahuge/orgs","repos_url":"https://api.github.com/users/Ahuge/repos","events_url":"https://api.github.com/users/Ahuge/events{/privacy}","received_events_url":"https://api.github.com/users/Ahuge/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-08-23T00:26:37Z","updated_at":"2017-08-23T15:49:39Z","closed_at":null,"body":"Hey,\r\n\r\nI'm going over some code that we are converting and it uses loadUiType instead of loadUi.\r\n\r\nIn the examples folder there is some code to implement a load_ui_type wrapper it looks  like.\r\n\r\nDo you have any plans for that code to make it's way into the core Qt.py module?"},"comment":{"url":"https://api.github.com/repos/mottosso/Qt.py/issues/comments/324378838","html_url":"https://github.com/mottosso/Qt.py/issues/237#issuecomment-324378838","issue_url":"https://api.github.com/repos/mottosso/Qt.py/issues/237","id":324378838,"user":{"login":"fredrikaverpil","id":994357,"avatar_url":"https://avatars2.githubusercontent.com/u/994357?v=4","gravatar_id":"","url":"https://api.github.com/users/fredrikaverpil","html_url":"https://github.com/fredrikaverpil","followers_url":"https://api.github.com/users/fredrikaverpil/followers","following_url":"https://api.github.com/users/fredrikaverpil/following{/other_user}","gists_url":"https://api.github.com/users/fredrikaverpil/gists{/gist_id}","starred_url":"https://api.github.com/users/fredrikaverpil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fredrikaverpil/subscriptions","organizations_url":"https://api.github.com/users/fredrikaverpil/orgs","repos_url":"https://api.github.com/users/fredrikaverpil/repos","events_url":"https://api.github.com/users/fredrikaverpil/events{/privacy}","received_events_url":"https://api.github.com/users/fredrikaverpil/received_events","type":"User","site_admin":false},"created_at":"2017-08-23T15:49:39Z","updated_at":"2017-08-23T15:49:39Z","body":"> Do you have any plans for that code to make it's way into the core Qt.py module?\r\n\r\nWe already support one way of importing .ui files which doesn't require `uic` or `pysideuic`; [`QtCompat.loadUi`](https://github.com/mottosso/Qt.py#compatibility). We'd like to steer users in this direction since e.g. older Nuke versions doesn't come with `pysideuic`. I'd be hesitant to implement `loadUiType` support, to be honest, and I would strongly suggest trying to use `QtCompat.loadUi` instead, if possible.\r\n\r\n@Ahuge However, I'd be happy to know more if you are unable to use `QtCompat.loadUi` for some reason.\r\n\r\nThose examples you found are \"leftovers\" from [implementing `Qt.load_ui`](https://github.com/mottosso/Qt.py/pulls?utf8=✓&q=is%3Apr%20qt.load_ui%20) (now `QtCompat.loadUi`). Initially, we didn't want to maintain such code but still wanted to provide some alternatives to users who wanted to load .ui files, and so we kept them in the \"examples\" folder.\r\n\r\nI see we should update the README for `loadUi`:\r\n\r\n> mapped to a convenience function in Qt.py called loadUi (part of the QtCompat module; QtCompat.loadUi). This function only accepts the filename argument of the .ui file.\r\n\r\nThis is [no longer true](https://github.com/mottosso/Qt.py/blob/master/Qt.py#L1180), as it does indeed support a `baseinstance` argument as well.\r\n"}}
{"actor":{"display_login":"fredrikaverpil"}}