Merge pull request #21 from bransoned/feature/17-error-handling-and-testing

#17 check attempts for download retry if server error

Author: bransoned

script.py

@@ -1003,8 +1003,10 @@ def memory_download(memories: list[dict[str, str, str, str, str]]) -> None:
                 # Retry on server errors. This seems to be most prevalent error when downloading
                 status = e.response.status_code
                 if 500 <= status < 600:
-                    print(f"\nMemory {idx}: Server error {status}, retry attempt {attempt}/{max_retries}")
-                    continue
+                    if attempt < max_retries:
+                        print(f"\nMemory {idx}: Server error {status}, retry attempt {attempt}/{max_retries}")
+                        time.sleep(retry_delay)
+                        continue
 
                 print(f"\nMemory {idx}: HTTP error {e.response.status_code}, skipping\n")
                 failed_downloads.append((idx, f"HTTP {e.response.status_code}"))