Merge pull request #2672 from alicevision/fix/externalMinFileWatcher

[ui] Graph: In minimal refresh, do not poll files for chunks run locally

Author: yann-lty

meshroom/ui/graph.py

@@ -38,7 +38,7 @@
 class PollerRefreshStatus(Enum):
     AUTO_ENABLED = 0  # The file watcher polls every single status file periodically
     DISABLED = 1  # The file watcher is disabled and never polls any file
-    MINIMAL_ENABLED = 2  # The file watcher only polls status files for chunks that are either submitted or running
+    MINIMAL_ENABLED = 2  # The file watcher only polls status files for chunks that are either submitted or running externally
 
 
 class FilesModTimePollerThread(QObject):
@@ -183,10 +183,12 @@ def watchedStatusFiles(self):
             return self.statusFiles, self.monitorableChunks
         elif self.filePollerRefresh is PollerRefreshStatus.MINIMAL_ENABLED.value:
             for c in self.monitorableChunks:
-                # When a chunk's status is ERROR, it may be externally re-submitted and it should thus still be monitored
-                if c._status.status is Status.SUBMITTED or c._status.status is Status.RUNNING or c._status.status is Status.ERROR:
-                    files.append(c.statusFile)
-                    chunks.append(c)
+                # Only chunks that are run externally should be monitored; when run locally, status changes are already notified
+                if c.isExtern():
+                    # Chunks with an ERROR status may be re-submitted externally and should thus still be monitored
+                    if c._status.status is Status.SUBMITTED or c._status.status is Status.RUNNING or c._status.status is Status.ERROR:
+                        files.append(c.statusFile)
+                        chunks.append(c)
         return files, chunks
 
     def compareFilesTimes(self, times):