From 09812019b561f2fc60504821fb9a7ff7b9dcc2f2 Mon Sep 17 00:00:00 2001
From: tcely <tcely@users.noreply.github.com>
Date: Fri, 7 Feb 2025 04:28:52 -0500
Subject: [PATCH] Use `next_progress` to avoid logging in spurts

---
 tubesync/sync/hooks.py | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/tubesync/sync/hooks.py b/tubesync/sync/hooks.py
index 27133b49..e3f55557 100644
--- a/tubesync/sync/hooks.py
+++ b/tubesync/sync/hooks.py
@@ -13,6 +13,7 @@ postprocessor_hook = {
     'status': dict(),
 }
 
+
 class BaseStatus:
     valid = set()
 
@@ -55,6 +56,11 @@ class ProgressHookStatus(BaseStatus):
         self.status = status
         self.download_progress = 0
 
+    def next_progress(self):
+        if 0 == self.download_progress:
+            return 0
+        return 5 + self.download_progress
+
 class PPHookStatus(BaseStatus):
     valid = frozenset((
         'started',
@@ -109,10 +115,10 @@ def yt_dlp_progress_hook(event):
             pass
         if downloaded_bytes > 0 and total_bytes > 0:
             percent = round(100 * downloaded_bytes / total_bytes)
-        if percent and (0 < percent) and (0 == percent % 5):
+        if percent and (self.next_progress() < percent) and (0 == percent % 5):
+            status.download_progress = percent
             log.info(f'[youtube-dl] downloading: {filename} - {percent_str} '
                      f'of {total} at {speed}, {eta} remaining')
-        status.download_progress = percent or 0
     elif 'finished' == event['status']:
         # update the status for key to the finished value
         status = ProgressHookStatus.get(key)