fotolab 0.29.1__py3-none-any.whl → 0.30.0__py3-none-any.whl

fotolab/__init__.py

@@ -24,7 +24,7 @@ from pathlib import Path
 
 from PIL import Image
 
-__version__ = "0.29.1"
+__version__ = "0.30.0"
 
 log = logging.getLogger(__name__)
 

fotolab/cli.py

@@ -15,9 +15,9 @@
 
 """A console program to manipulate photos.
 
-  website: https://github.com/kianmeng/fotolab
-  changelog: https://github.com/kianmeng/fotolab/blob/master/CHANGELOG.md
-  issues: https://github.com/kianmeng/fotolab/issues
+website: https://github.com/kianmeng/fotolab
+changelog: https://github.com/kianmeng/fotolab/blob/master/CHANGELOG.md
+issues: https://github.com/kianmeng/fotolab/issues
 """
 
 import argparse
@@ -143,7 +143,7 @@ def main(args: Optional[Sequence[str]] = None) -> None:
                 # correctly
                 log.error(
                     "subcommand '%s' is missing its execution function.",
-                    parsed_args.command
+                    parsed_args.command,
                 )
                 parser.print_help(sys.stderr)
                 raise SystemExit(1)

fotolab/subcommands/animate.py

@@ -26,6 +26,21 @@ from fotolab import _open_image
 log = logging.getLogger(__name__)
 
 
+def _validate_duration(value: str) -> int:
+    """Validate that the duration is a positive integer."""
+    try:
+        ivalue = int(value)
+        if ivalue <= 0:
+            raise argparse.ArgumentTypeError(
+                f"duration must be a positive integer, but got {value}"
+            )
+        return ivalue
+    except ValueError as e:
+        raise argparse.ArgumentTypeError(
+            f"duration must be an integer, but got '{value}'"
+        ) from e
+
+
 def build_subparser(subparsers) -> None:
     """Build the subparser."""
     animate_parser = subparsers.add_parser("animate", help="animate an image")
@@ -56,9 +71,9 @@ def build_subparser(subparsers) -> None:
         "-d",
         "--duration",
         dest="duration",
-        type=int,
+        type=_validate_duration,
         default=2500,
-        help="set the duration in milliseconds (default: '%(default)s')",
+        help="set the duration in milliseconds (must be a positive integer, default: '%(default)s')",
         metavar="DURATION",
     )
 
@@ -81,6 +96,34 @@ def build_subparser(subparsers) -> None:
         help="open the image using default program (default: '%(default)s')",
     )
 
+    animate_parser.add_argument(
+        "--webp-quality",
+        dest="webp_quality",
+        type=int,
+        default=80,
+        choices=range(0, 101),
+        help="set WEBP quality (0-100, default: '%(default)s')",
+        metavar="QUALITY",
+    )
+
+    animate_parser.add_argument(
+        "--webp-lossless",
+        dest="webp_lossless",
+        default=False,
+        action="store_true",
+        help="enable WEBP lossless compression (default: '%(default)s')",
+    )
+
+    animate_parser.add_argument(
+        "--webp-method",
+        dest="webp_method",
+        type=int,
+        default=4,
+        choices=range(0, 7),
+        help="set WEBP encoding method (0=fast, 6=slow/best, default: '%(default)s')",
+        metavar="METHOD",
+    )
+
     animate_parser.add_argument(
         "-od",
         "--output-dir",
@@ -101,31 +144,50 @@ def run(args: argparse.Namespace) -> None:
     """
     log.debug(args)
 
-    first_image = args.image_filenames[0]
-    animated_image = Image.open(first_image)
-
-    append_images = []
-    for image_filename in args.image_filenames[1:]:
-        append_images.append(Image.open(image_filename))
-
-    image_file = Path(first_image)
-    new_filename = Path(
-        args.output_dir,
-        image_file.with_name(f"animate_{image_file.stem}.{args.format}"),
-    )
-    new_filename.parent.mkdir(parents=True, exist_ok=True)
-
-    log.info("animate image: %s", new_filename)
-
-    animated_image.save(
-        new_filename,
-        format=args.format,
-        append_images=append_images,
-        save_all=True,
-        duration=args.duration,
-        loop=args.loop,
-        optimize=True,
-    )
+    first_image_filepath = args.image_filenames[0]
+    main_frame = None
+    other_frames = []
+
+    try:
+        main_frame = Image.open(first_image_filepath)
+
+        for image_filename in args.image_filenames[1:]:
+            img = Image.open(image_filename)
+            other_frames.append(img)
+
+        image_file = Path(first_image_filepath)
+        new_filename = Path(
+            args.output_dir,
+            image_file.with_name(f"animate_{image_file.stem}.{args.format}"),
+        )
+        new_filename.parent.mkdir(parents=True, exist_ok=True)
+
+        log.info("animate image: %s", new_filename)
+
+        save_kwargs = {
+            "format": args.format,
+            "append_images": other_frames,
+            "save_all": True,
+            "duration": args.duration,
+            "loop": args.loop,
+            "optimize": True,  # General optimization, good for GIF
+        }
+
+        if args.format == "webp":
+            save_kwargs["quality"] = args.webp_quality
+            save_kwargs["lossless"] = args.webp_lossless
+            save_kwargs["method"] = args.webp_method
+            # Pillow's WEBP save doesn't use a general 'optimize' like GIF.
+            # Specific WEBP params like 'method' and 'quality' control this.
+            # We can remove 'optimize' if it causes issues or is ignored for WEBP.
+            # For now, let's keep it, Pillow might handle it or ignore it.
+
+        main_frame.save(new_filename, **save_kwargs)
+    finally:
+        if main_frame:
+            main_frame.close()
+        for frame in other_frames:
+            frame.close()
 
     if args.open:
         _open_image(new_filename)

fotolab/subcommands/contrast.py

@@ -31,7 +31,9 @@ def _validate_cutoff(value: str) -> float:
     try:
         f_value = float(value)
     except ValueError as e:
-        raise argparse.ArgumentTypeError(f"invalid float value: '{value}'") from e
+        raise argparse.ArgumentTypeError(
+            f"invalid float value: '{value}'"
+        ) from e
     if not 0 <= f_value <= 50:
         raise argparse.ArgumentTypeError(
             f"cutoff value {f_value} must be between 0 and 50"

fotolab/subcommands/halftone.py

@@ -141,11 +141,13 @@ def _draw_halftone_dot(
     x = min(int(cell.col * cell.cellsize + cell.cellsize / 2), img_width - 1)
     y = min(int(cell.row * cell.cellsize + cell.cellsize / 2), img_height - 1)
 
-    # Ensure coordinates are non-negative (shouldn't happen with current logic, but safe)
+    # Ensure coordinates are non-negative (shouldn't happen with current logic,
+    # but safe)
     x = max(0, x)
     y = max(0, y)
 
-    # Get pixel value (brightness or color) from the source image using clamped coordinates
+    # Get pixel value (brightness or color) from the source image using clamped
+    # coordinates
     pixel_value = source_image.getpixel((x, y))
 
     if grayscale:
@@ -160,8 +162,8 @@ def _draw_halftone_dot(
         )
         dot_fill = pixel_value  # Use original color for color dots
 
-    # Calculate dot radius relative to cell size based on brightness
-    # Max radius is half the cell size. Scale by brightness (0-255).
+    # Calculate dot radius relative to cell size based on brightness Max radius
+    # is half the cell size. Scale by brightness (0-255).
     dot_radius = (brightness / 255.0) * (cell.cellsize / 2)
 
     # Draw the dot

fotolab/subcommands/info.py

@@ -33,7 +33,6 @@ def build_subparser(subparsers) -> None:
         dest="image_filename",
         help="set the image filename",
         type=str,
-        default=None,
         metavar="IMAGE_FILENAME",
     )
 
@@ -74,29 +73,22 @@ def run(args: argparse.Namespace) -> None:
     """
     log.debug(args)
 
-    # TODO: Add error handling for file open
-    # TODO: Use context manager `with Image.open(...)`
-    image = Image.open(args.image_filename)
+    with Image.open(args.image_filename) as image:
+        exif_tags = extract_exif_tags(image, args.sort)
 
-    exif_tags = extract_exif_tags(image, args.sort)
+        if not exif_tags:
+            print("No metadata found!")
+            return
 
-    if not exif_tags:
-        print("No metadata found!")
-        # Close the image if opened outside a 'with' block
-        image.close()
-        return
-
-    output_info = []
+        output_info = []
     specific_info_requested = False
 
     if args.camera:
         specific_info_requested = True
-        # TODO: Add error handling for missing keys
         output_info.append(camera_metadata(exif_tags))
 
     if args.datetime:
         specific_info_requested = True
-        # TODO: Add error handling for missing keys
         output_info.append(datetime(exif_tags))
 
     if specific_info_requested:
@@ -107,9 +99,6 @@ def run(args: argparse.Namespace) -> None:
         for tag_name, tag_value in exif_tags.items():
             print(f"{tag_name:<{tag_name_width}}: {tag_value}")
 
-    # Close the image if opened outside a 'with' block
-    image.close()
-
 
 def extract_exif_tags(image: Image.Image, sort: bool = False) -> dict:
     """Extract Exif metadata from image."""
@@ -135,13 +124,11 @@ def extract_exif_tags(image: Image.Image, sort: bool = False) -> dict:
 
 def datetime(exif_tags: dict):
     """Extract datetime metadata."""
-    # TODO: Add error handling for missing key
-    return exif_tags["DateTime"]
+    return exif_tags.get("DateTime", "Not available")
 
 
 def camera_metadata(exif_tags: dict):
     """Extract camera and model metadata."""
-    # TODO: Add error handling for missing keys
     make = exif_tags.get("Make", "")
     model = exif_tags.get("Model", "")
     metadata = f"{make} {model}"

{fotolab-0.29.1.dist-info → fotolab-0.30.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: fotolab
-Version: 0.29.1
+Version: 0.30.0
 Summary: A console program that manipulate images.
 Keywords: photography,photo
 Author-email: Kian-Meng Ang <kianmeng@cpan.org>
@@ -106,7 +106,8 @@ fotolab animate -h
 
 ```console
 usage: fotolab animate [-h] [-f FORMAT] [-d DURATION] [-l LOOP] [-op]
-                       [-od OUTPUT_DIR]
+                       [--webp-quality QUALITY] [--webp-lossless]
+                       [--webp-method METHOD] [-od OUTPUT_DIR]
                        IMAGE_FILENAMES [IMAGE_FILENAMES ...]
 
 positional arguments:
@@ -116,10 +117,16 @@ options:
   -h, --help            show this help message and exit
   -f, --format FORMAT   set the image format (default: 'gif')
   -d, --duration DURATION
-                        set the duration in milliseconds (default: '2500')
+                        set the duration in milliseconds (must be a positive
+                        integer, default: '2500')
   -l, --loop LOOP       set the loop cycle (default: '0')
   -op, --open           open the image using default program (default:
                         'False')
+  --webp-quality QUALITY
+                        set WEBP quality (0-100, default: '80')
+  --webp-lossless       enable WEBP lossless compression (default: 'False')
+  --webp-method METHOD  set WEBP encoding method (0=fast, 6=slow/best,
+                        default: '4')
   -od, --output-dir OUTPUT_DIR
                         set default output folder (default: 'output')
 ```

{fotolab-0.29.1.dist-info → fotolab-0.30.0.dist-info}/RECORD

@@ -1,21 +1,21 @@
-fotolab/__init__.py,sha256=KwMFpmP-DDvkiBIx6zr9XGWSONGzZo5Yr0aHKjYUmlg,3262
+fotolab/__init__.py,sha256=RNGEVnKSQe-y5yy2fi65zOV5cJGZk9tw9o7id0xBMX4,3262
 fotolab/__main__.py,sha256=aboOURPs_snOXTEWYR0q8oq1UTY9e-NxCd1j33V0wHI,833
-fotolab/cli.py,sha256=NH_u73SJhzwDJSRqpi1I4dc8wBfcqbDxNne3cio163A,4411
+fotolab/cli.py,sha256=oFiQXmsu3wIsM_DpZnL4B94sAoB62L16Am-cjxGmosY,4406
 fotolab/subcommands/__init__.py,sha256=l3DlIaJ3u3jGjnC1H1yV8LZ_nPqOLJ6gikD4BCaMAQ0,1129
-fotolab/subcommands/animate.py,sha256=OcS6Q6JM1TW6HF0dSgBsV9mpEGwDMcJlQssupkf0_ZA,3393
+fotolab/subcommands/animate.py,sha256=vmviz3cLnHfVENxFKiTimhx8nmbGbzumOP6dUd_UiUI,5524
 fotolab/subcommands/auto.py,sha256=ia-xegV1Z4HvYsbKgmTzf1NfNFdTDPWfZe7vQ1_90Ik,2425
 fotolab/subcommands/border.py,sha256=BS3BHytdWiNumxdKulKYK-WigbsKtPxECdvInUhUjSQ,4608
-fotolab/subcommands/contrast.py,sha256=ZHTvAJhRYZjNTrkHRZq3O6wba3LS7QjH1BfiGf4ZvGY,3005
+fotolab/subcommands/contrast.py,sha256=fcXmHnxDw74j5ZUDQ5cwWh0N4tpyqqvEjymnpITgrEk,3027
 fotolab/subcommands/env.py,sha256=QoxRvzZKgmoHTUxDV4QYhdChCpMWs5TbXFY_qIpIQpE,1469
-fotolab/subcommands/halftone.py,sha256=YOGtdl6N3aC4E47ac1rGzHU0HczVsujKNIBMyiRf2-c,6681
-fotolab/subcommands/info.py,sha256=EYuU06-rrnZZMY0XKwaIiqLypbd6x9KmCydiB1UhtTU,4010
+fotolab/subcommands/halftone.py,sha256=lt6RV0OuZkGs1LigTC1EcCCY42CocPFHWaJTDjJ5LFM,6693
+fotolab/subcommands/info.py,sha256=vie578cEOesAyHohm0xCWwZBqXpblYJtdHGSyBgxsoU,3581
 fotolab/subcommands/montage.py,sha256=d_3EcyRSFS8fKkczlHO8IRP-mrKhQUtkQndjfd0MKsc,2566
 fotolab/subcommands/resize.py,sha256=UOb2rg_5ArRj0gxPTDOww_ix3tqJRC0W5b_S-Lt1G4w,5444
 fotolab/subcommands/rotate.py,sha256=uBFjHyjiBSQLtrtH1p9myODIHUDr1gkL4PpU-6Y1Ofo,2575
 fotolab/subcommands/sharpen.py,sha256=YNho2IPbc-lPvSy3Bsjehc2JOEy27LPqFSGRULs9MyY,3492
 fotolab/subcommands/watermark.py,sha256=qRGUp1Lc22fZSJDFRqQGPiz8RSB293ebvOVTdsDLUE4,10351
-fotolab-0.29.1.dist-info/entry_points.txt,sha256=mvw7AY_yZkIyjAxPtHNed9X99NZeLnMxEeAfEJUbrCM,44
-fotolab-0.29.1.dist-info/LICENSE.md,sha256=tGtFDwxWTjuR9syrJoSv1Hiffd2u8Tu8cYClfrXS_YU,31956
-fotolab-0.29.1.dist-info/WHEEL,sha256=EZbGkh7Ie4PoZfRQ8I0ZuP9VklN_TvcZ6DSE5Uar4z4,81
-fotolab-0.29.1.dist-info/METADATA,sha256=0jUgjFRshumTtyFXLKe6_sQVnpyGLNez-Fl-KB5vHMk,12980
-fotolab-0.29.1.dist-info/RECORD,,
+fotolab-0.30.0.dist-info/entry_points.txt,sha256=mvw7AY_yZkIyjAxPtHNed9X99NZeLnMxEeAfEJUbrCM,44
+fotolab-0.30.0.dist-info/LICENSE.md,sha256=tGtFDwxWTjuR9syrJoSv1Hiffd2u8Tu8cYClfrXS_YU,31956
+fotolab-0.30.0.dist-info/WHEEL,sha256=EZbGkh7Ie4PoZfRQ8I0ZuP9VklN_TvcZ6DSE5Uar4z4,81
+fotolab-0.30.0.dist-info/METADATA,sha256=tPOkSTJz_hy0seegQ60vrMKDyAcbwZWcIBwPl0ZU5Ac,13395
+fotolab-0.30.0.dist-info/RECORD,,