Automatic thumbnail generation on publishing review

[Dreamink-Official]

Added product family "review" and representation to instance

Changelog Description

As proposed in ynput/ayon-core#1049 I adjusted the ExtractActiveViewThumbnail class to allow to extract a thumbnail for the product family "review" it adds a representation for the extracted thumbnail.

Testing notes:

1. Open Houdini and publish a review

[MustafaJafar]

Suggested change

	instance.data.setdefault("representations", [])
	representation = {
	"name": "thumbnail",
	"ext": "jpg",
	"files": os.path.basename(view_thumbnail),
	"stagingDir": os.path.dirname(view_thumbnail),
	"thumbnail": True
	}
	# Adding representation to instance
	instance.data["representations"].append(representation)
	self.log.debug(
	"Adding thumbnail representation: {}".format(representation)
	)

you shouldn't add representation for the thumbnail as there are dedicated plugins for creating the representation.

• extract_thumbnail_from_source.py
• extract_thumbnail.py

[Dreamink-Official]

Great! I didn’t know there were more general plugins in the core addon. That would’ve saved me some time figuring out how to add the thumbnail (even though it’s pretty easy, haha). So, creating a PR in the ayon-core repo to implement the 'review' family and 'houdini' as host is the right approach, correct?"

class ExtractThumbnail(pyblish.api.InstancePlugin):
    """Create jpg thumbnail from sequence using ffmpeg"""

    label = "Extract Thumbnail"
    order = pyblish.api.ExtractorOrder + 0.49
    families = [
        "imagesequence", "render", "render2d", "prerender",
        "source", "clip", "take", "online", "image", "review"
    ]
    hosts = [
        "shell",
        "fusion",
        "resolve",
        "traypublisher",
        "substancepainter",
        "nuke",
        "aftereffects",
        "unreal",
        "houdini",
    ]
    enabled = False

[MustafaJafar]

yes!
Houdini host is missing as you have pointed out.

[MustafaJafar]

My feeling, this is more of a "workaround" than a "proper solution".

Additionally, a snapshot from the sceneview may not look the same as the first frame of the review.
This is not for review only but also render and image sequence products.

What about using the first frame as the thumbnail? (we'll need to convert it to png or jpg).
My feeling, it can be a core feature for all DCCs. It's actually a core feature...

[Dreamink-Official]

Using the first frame as the thumbnail could also work. The input from our artists was that it would be great to be able to choose a specific frame. But you are right. In the end that will not look like the real render by using the screenshot of the viewport. So perhaps adding a option in the publisher to be able to select which frame should be used as thumbnail (not only the middle frame of the sequence) could be an option?

[MustafaJafar]

feel free to create an issue for it on ayon-core.

[MustafaJafar]

From previous discussion, the issue was not an implemented feature in Houdini.
The feature already exists in core addon but it was lacking Houdini host.

Closing this PR in favor of ynput/ayon-core#1048