add discover_gui (without introducing ui dependencies !) #389
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem:
Pyblish base let's us register guis with
pyblish.api.register_gui
, but not show them. 😞Each dcc implementation of pyblish has a show function, but pyblish base does not.
e.g. in pyblish-maya we can do:
but in pyblish-base we can't do:
show()
launches the registered UI, and on fail falls back to the next registered UI.However this code is duplicated across each dcc implementation, and not available in pyblish base.
Moving this code to base would make it easier to make dcc implementations, and enable devs to use this functionality for non dcc plugins.
Solution
this PR implements the discover_gui method from pyblish_maya & pyblish_3dsmax in pyblish base, without creating a UI dependency
now we can do this
Note
pyblish.show()
Pyblish base already allows us to show both default qml & lite explicitly.