From 361d21eff6afb403781e92fa32fc6588af0a9257 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Sun, 31 Dec 2023 14:20:37 +0200
Subject: [PATCH 01/14] =?UTF-8?q?=F0=9F=93=B8=20allow=20taking=20snapshoot?=
 =?UTF-8?q?s=20from=20houdini?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 openpype/hosts/houdini/api/lib.py | 70 +++++++++++++++++++++++++++++++
 1 file changed, 70 insertions(+)

diff --git a/openpype/hosts/houdini/api/lib.py b/openpype/hosts/houdini/api/lib.py
index edd50f10c1..7b09d2baf2 100644
--- a/openpype/hosts/houdini/api/lib.py
+++ b/openpype/hosts/houdini/api/lib.py
@@ -1054,3 +1054,73 @@ def add_self_publish_button(node):
     template = node.parmTemplateGroup()
     template.insertBefore((0,), button_parm)
     node.setParmTemplateGroup(template)
+
+
+def getSceneViewer():
+    """
+    return an instance of a visible viewport.
+    There may be many, some could be closed, any visible are current
+    """
+    panes = hou.ui.paneTabs()
+    panes = [x for x in panes if x.type() == hou.paneTabType.SceneViewer]
+    panes = sorted(panes, key=lambda x: x.isCurrentTab())
+    if panes:
+        return panes[-1]
+    else:
+        return None
+
+
+def sceneview_snapshoot(
+        filename="$HIP/thumbnails/$HIPNAME.$F4.jpg",
+        fstart=None,
+        fend=None
+    ):
+    """
+    It takes snapshoot of your scene view fot the given frame range.
+    So, it's capable of generating snapshoots image sequence.
+    It works in different Houdini context e.g. Objects, Solaris
+
+    Example:
+        from openpype.hosts.houdini.api import lib
+        lib.sceneview_snapshoot()
+
+    Notes:
+        .png output will render poorly, so use .jpg.
+
+        How it works:
+            Get the current sceneviewer (may be more than one or hidden)
+            and screengrab the perspective viewport to a file in the
+            publish location to be picked up with the publish.
+
+        Credits:
+            https://www.sidefx.com/forum/topic/42808/?page=1#post-354796
+
+    Args:
+        filename (str): filename with the desired full path.
+        fstart (int): the frame at which snapshooting starts
+        fend (int): the frame at which snapshooting ends
+    """
+
+    try:
+
+        if fstart is None:
+            fstart = hou.frame()
+        if fend is None:
+            fend = fstart = hou.frame()
+
+        sceneview = getSceneViewer()
+        if not sceneview:
+            log.debug("No SceneViewers detected.")
+            return
+        viewport = sceneview.curViewport()
+
+        # this will open an mplay window to show the result
+        flip_settings = sceneview.flipbookSettings().stash()
+        flip_settings.frameRange( (fstart, fend) )
+        flip_settings.output(filename)
+        flip_settings.outputToMPlay(False)
+        sceneview.flipbook(viewport, flip_settings)
+        log.debug("A snap of sceneview has been saved to: {}".format(filename))
+
+    except Exception as e:
+        log.debug(e)

From 5658b109259a55a93edbb30f2e0c7f35c64a6201 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Sun, 31 Dec 2023 14:31:31 +0200
Subject: [PATCH 02/14] =?UTF-8?q?=F0=9F=90=BE=20Resolve=20Hound?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 openpype/hosts/houdini/api/lib.py | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/openpype/hosts/houdini/api/lib.py b/openpype/hosts/houdini/api/lib.py
index 7b09d2baf2..1b172595c8 100644
--- a/openpype/hosts/houdini/api/lib.py
+++ b/openpype/hosts/houdini/api/lib.py
@@ -1073,8 +1073,7 @@ def getSceneViewer():
 def sceneview_snapshoot(
         filename="$HIP/thumbnails/$HIPNAME.$F4.jpg",
         fstart=None,
-        fend=None
-    ):
+        fend=None):
     """
     It takes snapshoot of your scene view fot the given frame range.
     So, it's capable of generating snapshoots image sequence.
@@ -1116,7 +1115,7 @@ def sceneview_snapshoot(
 
         # this will open an mplay window to show the result
         flip_settings = sceneview.flipbookSettings().stash()
-        flip_settings.frameRange( (fstart, fend) )
+        flip_settings.frameRange((fstart, fend))
         flip_settings.output(filename)
         flip_settings.outputToMPlay(False)
         sceneview.flipbook(viewport, flip_settings)

From ab61e755e99773d3f4997c4fcbf27250e47d8262 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Wed, 3 Jan 2024 17:22:52 +0200
Subject: [PATCH 03/14] =?UTF-8?q?=F0=9F=93=9D=20Kuba's=20and=20Big=20Roy's?=
 =?UTF-8?q?=20comments?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 openpype/hosts/houdini/api/lib.py | 59 +++++++++++++++----------------
 1 file changed, 29 insertions(+), 30 deletions(-)

diff --git a/openpype/hosts/houdini/api/lib.py b/openpype/hosts/houdini/api/lib.py
index 1b172595c8..db9955694d 100644
--- a/openpype/hosts/houdini/api/lib.py
+++ b/openpype/hosts/houdini/api/lib.py
@@ -1056,7 +1056,7 @@ def add_self_publish_button(node):
     node.setParmTemplateGroup(template)
 
 
-def getSceneViewer():
+def get_scene_viewer():
     """
     return an instance of a visible viewport.
     There may be many, some could be closed, any visible are current
@@ -1066,12 +1066,13 @@ def getSceneViewer():
     panes = sorted(panes, key=lambda x: x.isCurrentTab())
     if panes:
         return panes[-1]
-    else:
-        return None
+
+    return None
 
 
-def sceneview_snapshoot(
-        filename="$HIP/thumbnails/$HIPNAME.$F4.jpg",
+def sceneview_snapshot(
+        sceneview,
+        filepath="$HIP/thumbnails/$HIPNAME.$F4.jpg",
         fstart=None,
         fend=None):
     """
@@ -1081,7 +1082,8 @@ def sceneview_snapshoot(
 
     Example:
         from openpype.hosts.houdini.api import lib
-        lib.sceneview_snapshoot()
+        sceneview = hou.ui.paneTabOfType(hou.paneTabType.SceneViewer)
+        lib.sceneview_snapshot(sceneview)
 
     Notes:
         .png output will render poorly, so use .jpg.
@@ -1095,31 +1097,28 @@ def sceneview_snapshoot(
             https://www.sidefx.com/forum/topic/42808/?page=1#post-354796
 
     Args:
-        filename (str): filename with the desired full path.
-        fstart (int): the frame at which snapshooting starts
-        fend (int): the frame at which snapshooting ends
+        sceneview (hou.SceneViewer): The scene view pane from which you want
+                                     to take a snapshot.
+        filepath (str): thumbnail filepath.
+        fstart (int): the frame at which snapshot starts
+        fend (int): the frame at which snapshot ends
     """
 
-    try:
+    if fstart is None:
+        fstart = hou.frame()
+    if fend is None:
+        fend = fstart
 
-        if fstart is None:
-            fstart = hou.frame()
-        if fend is None:
-            fend = fstart = hou.frame()
+    if not isinstance(sceneview, hou.SceneViewer):
+        log.debug("Wrong Input. {} is not of type hou.SceneViewer."
+                  .format(sceneview))
+        return
+    viewport = sceneview.curViewport()
 
-        sceneview = getSceneViewer()
-        if not sceneview:
-            log.debug("No SceneViewers detected.")
-            return
-        viewport = sceneview.curViewport()
-
-        # this will open an mplay window to show the result
-        flip_settings = sceneview.flipbookSettings().stash()
-        flip_settings.frameRange((fstart, fend))
-        flip_settings.output(filename)
-        flip_settings.outputToMPlay(False)
-        sceneview.flipbook(viewport, flip_settings)
-        log.debug("A snap of sceneview has been saved to: {}".format(filename))
-
-    except Exception as e:
-        log.debug(e)
+    # this will open an mplay window to show the result
+    flip_settings = sceneview.flipbookSettings().stash()
+    flip_settings.frameRange((fstart, fend))
+    flip_settings.output(filepath)
+    flip_settings.outputToMPlay(False)
+    sceneview.flipbook(viewport, flip_settings)
+    log.debug("A snap of sceneview has been saved to: {}".format(filepath))

From a27c7bf8fd99effd8374075b07e1f036467964a4 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Wed, 3 Jan 2024 17:40:49 +0200
Subject: [PATCH 04/14] =?UTF-8?q?=E2=9C=8F=EF=B8=8F=20=20fix=20typo?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 openpype/hosts/houdini/api/lib.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/openpype/hosts/houdini/api/lib.py b/openpype/hosts/houdini/api/lib.py
index db9955694d..7e07bce2d1 100644
--- a/openpype/hosts/houdini/api/lib.py
+++ b/openpype/hosts/houdini/api/lib.py
@@ -1076,8 +1076,8 @@ def sceneview_snapshot(
         fstart=None,
         fend=None):
     """
-    It takes snapshoot of your scene view fot the given frame range.
-    So, it's capable of generating snapshoots image sequence.
+    It takes snapshot of your scene view fot the given frame range.
+    So, it's capable of generating snapshots image sequence.
     It works in different Houdini context e.g. Objects, Solaris
 
     Example:
@@ -1121,4 +1121,4 @@ def sceneview_snapshot(
     flip_settings.output(filepath)
     flip_settings.outputToMPlay(False)
     sceneview.flipbook(viewport, flip_settings)
-    log.debug("A snap of sceneview has been saved to: {}".format(filepath))
+    log.debug("A snapshot of sceneview has been saved to: {}".format(filepath))

From b6a432d39363e00894ecb0f3fc4a95586d6d14c8 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Wed, 3 Jan 2024 21:50:48 +0200
Subject: [PATCH 05/14] =?UTF-8?q?=F0=9F=96=BC=EF=B8=8F=20=20Extract=20acti?=
 =?UTF-8?q?ve=20view=20as=20thumbnail?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../publish/extract_active_view_thumbnail.py  | 58 +++++++++++++++++++
 1 file changed, 58 insertions(+)
 create mode 100644 openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py

diff --git a/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py b/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
new file mode 100644
index 0000000000..40b0dac02e
--- /dev/null
+++ b/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
@@ -0,0 +1,58 @@
+import os
+
+import pyblish.api
+
+from openpype.pipeline import publish, registered_host
+from openpype.hosts.houdini.api import lib
+from openpype.hosts.houdini.api.pipeline import IS_HEADLESS
+
+import hou
+
+
+class ExtractActiveViewThumbnail(publish.Extractor):
+    """Set instance thumbnail to a screengrab of current active viewport.
+
+    This makes it so that if an instance does not have a thumbnail set yet that
+    it will get a thumbnail of the currently active view at the time of
+    publishing as a fallback.
+
+    """
+    order = pyblish.api.ExtractorOrder + 0.49
+    label = "Extract Active View Thumbnail"
+    families = ["workfile"]
+    hosts = ["houdini"]
+
+    def process(self, instance):
+        if IS_HEADLESS:
+            self.log.debug(
+                "Skip extraction of active view thumbnail, due to being in"
+                "headless mode."
+            )
+            return
+
+        thumbnail = instance.data.get("thumbnailPath")
+        self.log.debug(thumbnail)
+        if not thumbnail:
+            view_thumbnail = self.get_view_thumbnail(instance)
+            if not view_thumbnail:
+                return
+
+            self.log.debug("Setting instance thumbnail path to: {}".format(
+                view_thumbnail
+            ))
+            instance.data["thumbnailPath"] = view_thumbnail
+
+    def get_view_thumbnail(self, instance):
+
+        host = registered_host()
+        current_filepath = host.get_current_workfile()
+        if not current_filepath:
+            self.log.error("No current workfile path. Thumbnail generation skipped")
+            return
+
+        thumbnail_path = "{}_thumbnail.jpg".format(
+            current_filepath.rsplit('.', 1)[0])
+        sceneview = lib.get_scene_viewer()
+        lib.sceneview_snapshot(sceneview, thumbnail_path)
+
+        return thumbnail_path

From 08ccdfcea8593776f6898b659bf5c3d606b89770 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Wed, 3 Jan 2024 21:52:33 +0200
Subject: [PATCH 06/14] =?UTF-8?q?=F0=9F=A6=B4=20Resolve=20Hound?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../plugins/publish/extract_active_view_thumbnail.py       | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py b/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
index 40b0dac02e..b326f6918d 100644
--- a/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
+++ b/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
@@ -1,13 +1,9 @@
-import os
-
 import pyblish.api
 
 from openpype.pipeline import publish, registered_host
 from openpype.hosts.houdini.api import lib
 from openpype.hosts.houdini.api.pipeline import IS_HEADLESS
 
-import hou
-
 
 class ExtractActiveViewThumbnail(publish.Extractor):
     """Set instance thumbnail to a screengrab of current active viewport.
@@ -47,7 +43,8 @@ class ExtractActiveViewThumbnail(publish.Extractor):
         host = registered_host()
         current_filepath = host.get_current_workfile()
         if not current_filepath:
-            self.log.error("No current workfile path. Thumbnail generation skipped")
+            self.log.error("No current workfile path. "
+                           "Thumbnail generation skipped")
             return
 
         thumbnail_path = "{}_thumbnail.jpg".format(

From 0e7efc5da979d3cfdaa12cdfa31b926569ff2ea0 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Thu, 4 Jan 2024 18:56:59 +0200
Subject: [PATCH 07/14] Kuba's comments - Better Code

---
 openpype/hosts/houdini/api/lib.py              | 18 +++++++++---------
 .../publish/extract_active_view_thumbnail.py   | 17 ++++++-----------
 2 files changed, 15 insertions(+), 20 deletions(-)

diff --git a/openpype/hosts/houdini/api/lib.py b/openpype/hosts/houdini/api/lib.py
index 7e07bce2d1..318de5f538 100644
--- a/openpype/hosts/houdini/api/lib.py
+++ b/openpype/hosts/houdini/api/lib.py
@@ -1073,8 +1073,8 @@ def get_scene_viewer():
 def sceneview_snapshot(
         sceneview,
         filepath="$HIP/thumbnails/$HIPNAME.$F4.jpg",
-        fstart=None,
-        fend=None):
+        frame_start=None,
+        frame_end=None):
     """
     It takes snapshot of your scene view fot the given frame range.
     So, it's capable of generating snapshots image sequence.
@@ -1100,14 +1100,14 @@ def sceneview_snapshot(
         sceneview (hou.SceneViewer): The scene view pane from which you want
                                      to take a snapshot.
         filepath (str): thumbnail filepath.
-        fstart (int): the frame at which snapshot starts
-        fend (int): the frame at which snapshot ends
+        frame_start (int): the frame at which snapshot starts
+        frame_end (int): the frame at which snapshot ends
     """
 
-    if fstart is None:
-        fstart = hou.frame()
-    if fend is None:
-        fend = fstart
+    if frame_start is None:
+        frame_start = hou.frame()
+    if frame_end is None:
+        frame_end = frame_start
 
     if not isinstance(sceneview, hou.SceneViewer):
         log.debug("Wrong Input. {} is not of type hou.SceneViewer."
@@ -1117,7 +1117,7 @@ def sceneview_snapshot(
 
     # this will open an mplay window to show the result
     flip_settings = sceneview.flipbookSettings().stash()
-    flip_settings.frameRange((fstart, fend))
+    flip_settings.frameRange((frame_start, frame_end))
     flip_settings.output(filepath)
     flip_settings.outputToMPlay(False)
     sceneview.flipbook(viewport, flip_settings)
diff --git a/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py b/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
index b326f6918d..82976bdf83 100644
--- a/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
+++ b/openpype/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
@@ -1,6 +1,6 @@
 import pyblish.api
-
-from openpype.pipeline import publish, registered_host
+import tempfile
+from openpype.pipeline import publish
 from openpype.hosts.houdini.api import lib
 from openpype.hosts.houdini.api.pipeline import IS_HEADLESS
 
@@ -27,7 +27,6 @@ class ExtractActiveViewThumbnail(publish.Extractor):
             return
 
         thumbnail = instance.data.get("thumbnailPath")
-        self.log.debug(thumbnail)
         if not thumbnail:
             view_thumbnail = self.get_view_thumbnail(instance)
             if not view_thumbnail:
@@ -40,15 +39,11 @@ class ExtractActiveViewThumbnail(publish.Extractor):
 
     def get_view_thumbnail(self, instance):
 
-        host = registered_host()
-        current_filepath = host.get_current_workfile()
-        if not current_filepath:
-            self.log.error("No current workfile path. "
-                           "Thumbnail generation skipped")
-            return
+        with tempfile.NamedTemporaryFile("w", suffix=".jpg") as tmp:
+            thumbnail_path = tmp.name
+
+        instance.context.data["cleanupFullPaths"].append(thumbnail_path)
 
-        thumbnail_path = "{}_thumbnail.jpg".format(
-            current_filepath.rsplit('.', 1)[0])
         sceneview = lib.get_scene_viewer()
         lib.sceneview_snapshot(sceneview, thumbnail_path)
 

From 435cd9cce1e6a95fd5074aef1282866b8c126ef8 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Fri, 5 Jan 2024 17:04:36 +0200
Subject: [PATCH 08/14] =?UTF-8?q?=E2=9E=96=20Remove=20unnecessary=20commen?=
 =?UTF-8?q?t?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 openpype/hosts/houdini/api/lib.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/openpype/hosts/houdini/api/lib.py b/openpype/hosts/houdini/api/lib.py
index 318de5f538..bd15b8b83d 100644
--- a/openpype/hosts/houdini/api/lib.py
+++ b/openpype/hosts/houdini/api/lib.py
@@ -1115,7 +1115,6 @@ def sceneview_snapshot(
         return
     viewport = sceneview.curViewport()
 
-    # this will open an mplay window to show the result
     flip_settings = sceneview.flipbookSettings().stash()
     flip_settings.frameRange((frame_start, frame_end))
     flip_settings.output(filepath)

From 3ffb6db49fe161e1790c039142bc2236b492265a Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Tue, 9 Jan 2024 10:01:38 +0200
Subject: [PATCH 09/14] =?UTF-8?q?=F0=9F=93=9D=20Stick=20to=20Google=20doc?=
 =?UTF-8?q?=20string=20style?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 openpype/hosts/houdini/api/lib.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/openpype/hosts/houdini/api/lib.py b/openpype/hosts/houdini/api/lib.py
index bd15b8b83d..33b74ca6b9 100644
--- a/openpype/hosts/houdini/api/lib.py
+++ b/openpype/hosts/houdini/api/lib.py
@@ -1058,7 +1058,8 @@ def add_self_publish_button(node):
 
 def get_scene_viewer():
     """
-    return an instance of a visible viewport.
+    Return an instance of a visible viewport.
+
     There may be many, some could be closed, any visible are current
     """
     panes = hou.ui.paneTabs()
@@ -1075,8 +1076,9 @@ def sceneview_snapshot(
         filepath="$HIP/thumbnails/$HIPNAME.$F4.jpg",
         frame_start=None,
         frame_end=None):
-    """
-    It takes snapshot of your scene view fot the given frame range.
+    """Take a snapshot of your scene view.
+
+    It takes snapshot of your scene view for the given frame range.
     So, it's capable of generating snapshots image sequence.
     It works in different Houdini context e.g. Objects, Solaris
 

From 62020ca29d2ec75b7e545fbf3d1a0b835dd86e98 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Mon, 4 Mar 2024 15:46:28 +0200
Subject: [PATCH 10/14] update 'sceneview_snapshot' docstring

---
 client/ayon_core/hosts/houdini/api/lib.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/client/ayon_core/hosts/houdini/api/lib.py b/client/ayon_core/hosts/houdini/api/lib.py
index b9a92fab0c..3cab61ff26 100644
--- a/client/ayon_core/hosts/houdini/api/lib.py
+++ b/client/ayon_core/hosts/houdini/api/lib.py
@@ -1083,9 +1083,11 @@ def sceneview_snapshot(
     It works in different Houdini context e.g. Objects, Solaris
 
     Example:
-        from openpype.hosts.houdini.api import lib
-        sceneview = hou.ui.paneTabOfType(hou.paneTabType.SceneViewer)
-        lib.sceneview_snapshot(sceneview)
+    	This is how the function can be used::
+    	
+        	from ayon_core.hosts.houdini.api import lib
+	        sceneview = hou.ui.paneTabOfType(hou.paneTabType.SceneViewer)
+        	lib.sceneview_snapshot(sceneview)
 
     Notes:
         .png output will render poorly, so use .jpg.

From ab6d89c8c29c0dd883cbece387f8a9f872a3aaa3 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Mon, 4 Mar 2024 16:01:08 +0200
Subject: [PATCH 11/14] wrap lib.sceneview_snapshot() into
 tempfile.NamedTemporaryFile

---
 .../plugins/publish/extract_active_view_thumbnail.py      | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py b/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
index 27c4493f44..392201e865 100644
--- a/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
+++ b/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
@@ -39,12 +39,10 @@ class ExtractActiveViewThumbnail(publish.Extractor):
 
     def get_view_thumbnail(self, instance):
 
-        with tempfile.NamedTemporaryFile("w", suffix=".jpg") as tmp:
+        sceneview = lib.get_scene_viewer()
+        with tempfile.NamedTemporaryFile("w", suffix=".jpg", delete=False) as tmp:
+            lib.sceneview_snapshot(sceneview, tmp.name)
             thumbnail_path = tmp.name
 
         instance.context.data["cleanupFullPaths"].append(thumbnail_path)
-
-        sceneview = lib.get_scene_viewer()
-        lib.sceneview_snapshot(sceneview, thumbnail_path)
-
         return thumbnail_path

From 8320ab7b5138fdc57f352e708738cbce0f40c424 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Mon, 4 Mar 2024 16:46:34 +0200
Subject: [PATCH 12/14] Houdini: add a warning if no sceneview was found

---
 .../houdini/plugins/publish/extract_active_view_thumbnail.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py b/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
index 392201e865..1be8cb7440 100644
--- a/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
+++ b/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
@@ -40,6 +40,11 @@ class ExtractActiveViewThumbnail(publish.Extractor):
     def get_view_thumbnail(self, instance):
 
         sceneview = lib.get_scene_viewer()
+        if sceneview is None:
+            self.log.warning("Skipping Extract Active View Thumbnail"
+                             " because no scene view was detected.")
+            return
+
         with tempfile.NamedTemporaryFile("w", suffix=".jpg", delete=False) as tmp:
             lib.sceneview_snapshot(sceneview, tmp.name)
             thumbnail_path = tmp.name

From 71bb5552f38ea72e1bab07f41281cc3e595070ae Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Tue, 2 Apr 2024 12:30:33 +0200
Subject: [PATCH 13/14] BigRoy's comments - Better doc strings

---
 client/ayon_core/hosts/houdini/api/lib.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/client/ayon_core/hosts/houdini/api/lib.py b/client/ayon_core/hosts/houdini/api/lib.py
index dfdca62b61..111b230252 100644
--- a/client/ayon_core/hosts/houdini/api/lib.py
+++ b/client/ayon_core/hosts/houdini/api/lib.py
@@ -1003,6 +1003,9 @@ def get_scene_viewer():
     Return an instance of a visible viewport.
 
     There may be many, some could be closed, any visible are current
+
+    Returns:
+        Optional[hou.SceneViewer]: A scene viewer, if any.
     """
     panes = hou.ui.paneTabs()
     panes = [x for x in panes if x.type() == hou.paneTabType.SceneViewer]
@@ -1045,7 +1048,9 @@ def sceneview_snapshot(
     Args:
         sceneview (hou.SceneViewer): The scene view pane from which you want
                                      to take a snapshot.
-        filepath (str): thumbnail filepath.
+        filepath (str): thumbnail filepath. it expects `$F4` token
+                        when frame_end is bigger than frame_star other wise
+                        each frame will override its predecessor.
         frame_start (int): the frame at which snapshot starts
         frame_end (int): the frame at which snapshot ends
     """

From f543e560011a6bae6346e9a611ec1cc102fbb690 Mon Sep 17 00:00:00 2001
From: MustafaJafar <mustafataherzaky@outlook.com>
Date: Tue, 2 Apr 2024 12:36:22 +0200
Subject: [PATCH 14/14] BigRoy's comment - Better logic

---
 .../publish/extract_active_view_thumbnail.py  | 22 ++++++++++---------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py b/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
index 1be8cb7440..aedcb1da02 100644
--- a/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
+++ b/client/ayon_core/hosts/houdini/plugins/publish/extract_active_view_thumbnail.py
@@ -27,22 +27,24 @@ class ExtractActiveViewThumbnail(publish.Extractor):
             return
 
         thumbnail = instance.data.get("thumbnailPath")
-        if not thumbnail:
-            view_thumbnail = self.get_view_thumbnail(instance)
-            if not view_thumbnail:
-                return
+        if thumbnail:
+            # A thumbnail was already set for this instance
+            return
 
-            self.log.debug("Setting instance thumbnail path to: {}".format(
-                view_thumbnail
-            ))
-            instance.data["thumbnailPath"] = view_thumbnail
+        view_thumbnail = self.get_view_thumbnail(instance)
+        if not view_thumbnail:
+            return
+        self.log.debug("Setting instance thumbnail path to: {}"
+                       .format(view_thumbnail)
+        )
+        instance.data["thumbnailPath"] = view_thumbnail
 
     def get_view_thumbnail(self, instance):
 
         sceneview = lib.get_scene_viewer()
         if sceneview is None:
-            self.log.warning("Skipping Extract Active View Thumbnail"
-                             " because no scene view was detected.")
+            self.log.debug("Skipping Extract Active View Thumbnail"
+                           " because no scene view was detected.")
             return
 
         with tempfile.NamedTemporaryFile("w", suffix=".jpg", delete=False) as tmp: