mapillary-downloader 0.3.0__tar.gz → 0.3.1__tar.gz

{mapillary_downloader-0.3.0 → mapillary_downloader-0.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mapillary_downloader
-Version: 0.3.0
+Version: 0.3.1
 Summary: Download your Mapillary data before it's gone
 Author-email: Gareth Davidson <gaz@bitplane.net>
 Requires-Python: >=3.10
@@ -63,7 +63,7 @@ mapillary-downloader --token YOUR_TOKEN --username SOME_USERNAME --output ./down
 | `--quality`   | 256, 1024, 2048 or original           | `original`         |
 | `--bbox`      | `west,south,east,north`               | `None`             |
 | `--webp`      | Convert to WebP (saves ~70% space)    | `False`            |
-| `--workers`   | Number of parallel download workers   | CPU count          |
+| `--workers`   | Number of parallel download workers   | Half of CPU count  |
 | `--no-tar`    | Don't tar sequence directories        | `False`            |
 
 The downloader will:

{mapillary_downloader-0.3.0 → mapillary_downloader-0.3.1}/README.md

@@ -33,7 +33,7 @@ mapillary-downloader --token YOUR_TOKEN --username SOME_USERNAME --output ./down
 | `--quality`   | 256, 1024, 2048 or original           | `original`         |
 | `--bbox`      | `west,south,east,north`               | `None`             |
 | `--webp`      | Convert to WebP (saves ~70% space)    | `False`            |
-| `--workers`   | Number of parallel download workers   | CPU count          |
+| `--workers`   | Number of parallel download workers   | Half of CPU count  |
 | `--no-tar`    | Don't tar sequence directories        | `False`            |
 
 The downloader will:

{mapillary_downloader-0.3.0 → mapillary_downloader-0.3.1}/pyproject.toml

@@ -1,7 +1,7 @@
 [project]
 name = "mapillary_downloader"
 description = "Download your Mapillary data before it's gone"
-version = "0.3.0"
+version = "0.3.1"
 authors = [
     { name = "Gareth Davidson", email = "gaz@bitplane.net" }
 ]

{mapillary_downloader-0.3.0 → mapillary_downloader-0.3.1}/src/mapillary_downloader/__main__.py

@@ -38,7 +38,7 @@ def main():
         "--workers",
         type=int,
         default=None,
-        help="Number of parallel workers (default: number of CPU cores)",
+        help="Number of parallel workers (default: half of CPU cores)",
     )
     parser.add_argument(
         "--no-tar",

{mapillary_downloader-0.3.0 → mapillary_downloader-0.3.1}/src/mapillary_downloader/downloader.py

@@ -25,14 +25,14 @@ class MapillaryDownloader:
             output_dir: Base directory to save downloads
             username: Mapillary username (for collection directory)
             quality: Image quality (for collection directory)
-            workers: Number of parallel workers (default: cpu_count)
+            workers: Number of parallel workers (default: half of cpu_count)
             tar_sequences: Whether to tar sequence directories after download (default: True)
         """
         self.client = client
         self.base_output_dir = Path(output_dir)
         self.username = username
         self.quality = quality
-        self.workers = workers if workers is not None else os.cpu_count()
+        self.workers = workers if workers is not None else max(1, os.cpu_count() // 2)
         self.tar_sequences = tar_sequences
 
         # If username and quality provided, create collection directory

{mapillary_downloader-0.3.0 → mapillary_downloader-0.3.1}/src/mapillary_downloader/worker.py

@@ -4,7 +4,6 @@ import tempfile
 from pathlib import Path
 import requests
 from requests.exceptions import RequestException
-import time
 from mapillary_downloader.exif_writer import write_exif_to_image
 from mapillary_downloader.webp_converter import convert_to_webp
 
@@ -54,29 +53,23 @@ def download_and_convert_image(image_data, output_dir, quality, convert_webp, ac
             final_path = jpg_path
 
         # Download image
+        # No retries for CDN images - they're cheap, just skip failures and move on
         session = requests.Session()
         session.headers.update({"Authorization": f"OAuth {access_token}"})
 
-        max_retries = 10
-        base_delay = 1.0
         bytes_downloaded = 0
 
-        for attempt in range(max_retries):
-            try:
-                response = session.get(image_url, stream=True, timeout=60)
-                response.raise_for_status()
-
-                with open(jpg_path, "wb") as f:
-                    for chunk in response.iter_content(chunk_size=8192):
-                        f.write(chunk)
-                        bytes_downloaded += len(chunk)
-                break
-            except RequestException as e:
-                if attempt == max_retries - 1:
-                    return (image_id, 0, False, f"Download failed: {e}")
-
-                delay = base_delay * (2**attempt)
-                time.sleep(delay)
+        try:
+            # 60 second timeout for entire download (connection + read)
+            response = session.get(image_url, stream=True, timeout=60)
+            response.raise_for_status()
+
+            with open(jpg_path, "wb") as f:
+                for chunk in response.iter_content(chunk_size=8192):
+                    f.write(chunk)
+                    bytes_downloaded += len(chunk)
+        except RequestException as e:
+            return (image_id, 0, False, f"Download failed: {e}")
 
         # Write EXIF metadata
         write_exif_to_image(jpg_path, image_data)