scitex 2.3.0__py3-none-any.whl → 2.4.1__py3-none-any.whl

scitex/ai/classification/reporters/reporter_utils/_Plotter.py

@@ -44,7 +44,7 @@ try:
 
     # Import scitex plotting functions
     import scitex as stx
-    from scitex.ai.plt.plot_conf_mat import plot_conf_mat as conf_mat
+    from scitex.ai.plt.stx_conf_mat import stx_conf_mat as conf_mat
     from scitex.ai.plt.plot_roc_curve import plot_roc_curve as roc_auc
     from scitex.ai.plt.plot_pre_rec_curve import plot_pre_rec_curve as pre_rec_auc
 

scitex/ai/plt/__init__.py

@@ -6,7 +6,7 @@ but re-exported here for backward compatibility. New code should import directly
 from scitex.ai.metrics instead.
 """
 
-from ._plot_conf_mat import calc_bACC_from_conf_mat, calc_bacc_from_conf_mat, plot_conf_mat, conf_mat
+from ._stx_conf_mat import calc_bACC_from_conf_mat, calc_bacc_from_conf_mat, stx_conf_mat, conf_mat
 from ._plot_learning_curve import (
     plot_learning_curve,
     _prepare_metrics_df,
@@ -37,7 +37,7 @@ vline_at_epochs = _add_epoch_vlines
 
 __all__ = [
     # Plotting functions
-    "plot_conf_mat",
+    "stx_conf_mat",
     "conf_mat",  # backward compat
     "plot_learning_curve",
     "learning_curve",  # backward compat

scitex/ai/plt/{_plot_conf_mat.py → _stx_conf_mat.py}

@@ -1,7 +1,7 @@
 #!/usr/bin/env python3
 # -*- coding: utf-8 -*-
 # Timestamp: "2025-10-02 07:15:10 (ywatanabe)"
-# File: /ssh:sp:/home/ywatanabe/proj/scitex_repo/src/scitex/ml/plt/plot_conf_mat.py
+# File: /ssh:sp:/home/ywatanabe/proj/scitex_repo/src/scitex/ml/plt/stx_conf_mat.py
 # ----------------------------------------
 from __future__ import annotations
 import os
@@ -28,7 +28,7 @@ from scitex.ai.metrics import calc_bacc_from_conf_mat
 calc_bACC_from_conf_mat = calc_bacc_from_conf_mat
 
 
-def plot_conf_mat(
+def stx_conf_mat(
     cm=None,
     y_true=None,
     y_pred=None,
@@ -270,7 +270,7 @@ def plot_conf_mat(
 
 
 # Backward compatibility alias
-conf_mat = plot_conf_mat
+conf_mat = stx_conf_mat
 
 
 # Metric calculation functions have been moved to scitex.ai.metrics

scitex/config/PriorityConfig.py

@@ -0,0 +1,195 @@
+#!/usr/bin/env python3
+# -*- coding: utf-8 -*-
+# Timestamp: "2025-12-01 18:53:13 (ywatanabe)"
+# File: /home/ywatanabe/proj/scitex-code/src/scitex/config/PriorityConfig.py
+
+
+"""
+Priority-based configuration resolver.
+
+Provides clean precedence hierarchy: direct → env → config → default
+
+Based on priority-config by ywatanabe (https://github.com/ywatanabe1989/priority-config)
+Incorporated into scitex for self-contained configuration management.
+"""
+
+import os
+from typing import Dict
+from typing import List
+from typing import Optional, Type, Any
+
+
+class PriorityConfig:
+    """Universal config resolver with precedence: direct → env → config → default
+
+    Examples
+    --------
+    >>> from scitex.config import PriorityConfig
+    >>> config = PriorityConfig(config_dict={"port": 3000}, env_prefix="SCITEX_")
+    >>> port = config.resolve("port", None, default=8000, type=int)
+    3000  # from config_dict (env not set)
+    >>> # With env: PORT=5000 python script.py
+    >>> port = config.resolve("port", None, default=8000, type=int)
+    5000  # env takes precedence over config
+    >>> port = config.resolve("port", 9000, default=8000, type=int)
+    9000  # direct value takes highest precedence
+    """
+
+    SENSITIVE_EXPRESSIONS = [
+        "API",
+        "PASSWORD",
+        "SECRET",
+        "TOKEN",
+        "KEY",
+        "PASS",
+        "AUTH",
+        "CREDENTIAL",
+        "PRIVATE",
+        "CERT",
+    ]
+
+    def __init__(
+        self,
+        config_dict: Optional[Dict[str, Any]] = None,
+        env_prefix: str = "",
+        auto_uppercase: bool = True,
+    ):
+        """Initialize PriorityConfig.
+
+        Parameters
+        ----------
+        config_dict : dict, optional
+            Dictionary with configuration values
+        env_prefix : str
+            Prefix for environment variables (e.g., "SCITEX_")
+        auto_uppercase : bool
+            Whether to uppercase keys for env lookup
+        """
+        self.config_dict = config_dict or {}
+        self.env_prefix = env_prefix
+        self.auto_uppercase = auto_uppercase
+        self.resolution_log: List[Dict[str, Any]] = []
+
+    def __repr__(self) -> str:
+        return f"PriorityConfig(prefix='{self.env_prefix}', configs={len(self.config_dict)})"
+
+    def get(self, key: str) -> Any:
+        """Get value from config dict only."""
+        return self.config_dict.get(key)
+
+    def resolve(
+        self,
+        key: str,
+        direct_val: Any = None,
+        default: Any = None,
+        type: Type = str,
+        mask: Optional[bool] = None,
+    ) -> Any:
+        """Get value with precedence hierarchy.
+
+        Precedence: direct → env → config → default
+
+        Parameters
+        ----------
+        key : str
+            Configuration key to resolve
+        direct_val : Any, optional
+            Direct value (highest precedence)
+        default : Any, optional
+            Default value if not found elsewhere
+        type : Type
+            Type conversion (str, int, float, bool, list)
+        mask : bool, optional
+            Override automatic masking of sensitive values
+
+        Returns
+        -------
+        Any
+            Resolved configuration value
+        """
+        source = None
+        final_value = None
+
+        # Replace dots with underscores for env key (e.g., axes.width_mm -> AXES_WIDTH_MM)
+        normalized_key = key.replace(".", "_")
+        env_key = f"{self.env_prefix}{normalized_key.upper() if self.auto_uppercase else normalized_key}"
+        env_val = os.getenv(env_key)
+
+        # Priority: direct → env → config → default
+        if direct_val is not None:
+            source = "direct"
+            final_value = direct_val
+        elif env_val:
+            source = f"env:{env_key}"
+            final_value = self._convert_type(env_val, type)
+        elif key in self.config_dict:
+            source = "config"
+            final_value = self.config_dict[key]
+        else:
+            source = "default"
+            final_value = default
+
+        if mask is False:
+            should_mask = False
+        else:
+            should_mask = self._is_sensitive(key)
+
+        display_value = (
+            self._mask_value(final_value) if should_mask else final_value
+        )
+
+        self.resolution_log.append(
+            {
+                "key": key,
+                "source": source,
+                "value": display_value,
+                "type": type.__name__,
+            }
+        )
+
+        return final_value
+
+    def print_resolutions(self) -> None:
+        """Print how each config was resolved."""
+        if not self.resolution_log:
+            print("No configurations resolved yet")
+            return
+
+        print("Configuration Resolution Log:")
+        print("-" * 50)
+        for entry in self.resolution_log:
+            print(
+                f"{entry['key']:<20} = {entry['value']:<20} ({entry['source']})"
+            )
+
+    def clear_log(self) -> None:
+        """Clear resolution log."""
+        self.resolution_log = []
+
+    def _convert_type(self, value: str, type: Type) -> Any:
+        """Convert string value to specified type."""
+        if type == int:
+            return int(value)
+        elif type == float:
+            return float(value)
+        elif type == bool:
+            return value.lower() in ("true", "1", "yes")
+        elif type == list:
+            return value.split(",")
+        return value
+
+    def _is_sensitive(self, key: str) -> bool:
+        """Check if key contains sensitive expressions."""
+        key_upper = key.upper()
+        return any(expr in key_upper for expr in self.SENSITIVE_EXPRESSIONS)
+
+    def _mask_value(self, value: Any) -> str:
+        """Mask sensitive values for display."""
+        if value is None:
+            return None
+        value_str = str(value)
+        if len(value_str) <= 4:
+            return "****"
+        return value_str[:2] + "*" * (len(value_str) - 4) + value_str[-2:]
+
+# EOF

scitex/config/__init__.py

@@ -0,0 +1,24 @@
+#!/usr/bin/env python3
+# -*- coding: utf-8 -*-
+# Timestamp: "2025-12-01 21:00:00 (ywatanabe)"
+# File: ./src/scitex/config/__init__.py
+
+"""
+SciTeX configuration module.
+
+Provides priority-based configuration resolution with clean precedence:
+direct → config → env → default
+
+Usage:
+    from scitex.config import PriorityConfig
+
+    config = PriorityConfig(config_dict={"port": 3000}, env_prefix="SCITEX_")
+    port = config.resolve("port", None, default=8000, type=int)
+"""
+
+from .PriorityConfig import PriorityConfig
+
+__all__ = ["PriorityConfig"]
+
+
+# EOF

scitex/io/_save.py

@@ -157,7 +157,7 @@ def save(
     dry_run: bool = False,
     no_csv: bool = False,
     use_caller_path: bool = False,
-    auto_crop: bool = False,
+    auto_crop: bool = True,
     crop_margin_mm: float = 1.0,
     metadata_extra: dict = None,
     **kwargs,
@@ -183,7 +183,7 @@ def save(
         If True, simulate the saving process without actually writing files. Default is False.
     auto_crop : bool, optional
         If True, automatically crop the saved image to content area with margin (for PNG/JPEG/TIFF).
-        Vector formats (PDF/SVG) are not cropped. Default is False.
+        Vector formats (PDF/SVG) are not cropped. Default is True.
     crop_margin_mm : float, optional
         Margin in millimeters to add around content when auto_crop=True.
         At 300 DPI: 1mm = ~12 pixels. Default is 1.0mm (Nature Reviews style).
@@ -764,6 +764,13 @@ def _handle_image_with_csv(
         ext = _os.path.splitext(spath)[1].lower()
         ext_wo_dot = ext.replace(".", "")
 
+        # Check if the path contains an image extension directory (e.g., ./png/, ./jpg/)
+        # If so, save CSV in a parallel ./csv/ directory
+        image_extensions = ['png', 'jpg', 'jpeg', 'gif', 'tiff', 'tif', 'svg', 'pdf']
+        parent_dir = _os.path.dirname(spath)
+        parent_name = _os.path.basename(parent_dir)
+        filename_without_ext = _os.path.splitext(_os.path.basename(spath))[0]
+
         try:
             # Get the figure object that may contain plot data
             fig_obj = _get_figure_with_data(obj)
@@ -773,10 +780,18 @@ def _handle_image_with_csv(
                 if hasattr(fig_obj, "export_as_csv"):
                     csv_data = fig_obj.export_as_csv()
                     if csv_data is not None and not csv_data.empty:
-                        # Save CSV in same directory as image with .csv extension
-                        # Example: ./path/to/output/fig.jpg -> ./path/to/output/fig.csv
-                        csv_path = _os.path.splitext(spath)[0] + ".csv"
-                        # Ensure parent directory exists (should already exist from image save)
+                        # Determine CSV path based on parent directory name
+                        if parent_name.lower() in image_extensions:
+                            # Parent directory is named after an image extension (e.g., png/)
+                            # Create parallel csv/ directory
+                            grandparent_dir = _os.path.dirname(parent_dir)
+                            csv_dir = _os.path.join(grandparent_dir, "csv")
+                            csv_path = _os.path.join(csv_dir, filename_without_ext + ".csv")
+                        else:
+                            # Save CSV in same directory as image
+                            csv_path = _os.path.splitext(spath)[0] + ".csv"
+
+                        # Ensure parent directory exists
                         _os.makedirs(_os.path.dirname(csv_path), exist_ok=True)
                         # Save directly using _save to avoid path doubling
                         # Don't pass image-specific kwargs to CSV save
@@ -791,9 +806,19 @@ def _handle_image_with_csv(
 
                         # Create symlink_to for CSV if it was specified for the image
                         if symlink_to:
-                            csv_symlink_to = (
-                                _os.path.splitext(symlink_to)[0] + ".csv"
-                            )
+                            # Apply same directory transformation for symlink
+                            symlink_parent_dir = _os.path.dirname(symlink_to)
+                            symlink_parent_name = _os.path.basename(symlink_parent_dir)
+                            symlink_filename_without_ext = _os.path.splitext(_os.path.basename(symlink_to))[0]
+
+                            if symlink_parent_name.lower() in image_extensions:
+                                symlink_grandparent_dir = _os.path.dirname(symlink_parent_dir)
+                                csv_symlink_to = _os.path.join(
+                                    symlink_grandparent_dir, "csv", symlink_filename_without_ext + ".csv"
+                                )
+                            else:
+                                csv_symlink_to = _os.path.splitext(symlink_to)[0] + ".csv"
+
                             _symlink_to(csv_path, csv_symlink_to, True)
 
                         # Create symlink for CSV manually if needed
@@ -810,13 +835,22 @@ def _handle_image_with_csv(
                                         "specified_path"
                                     ]
                                     if isinstance(original_path, str):
-                                        # Replace extension to get CSV path structure
-                                        csv_relative = original_path.replace(
-                                            _os.path.splitext(original_path)[
-                                                1
-                                            ],
-                                            ".csv",
-                                        )
+                                        # Apply same directory transformation for symlink
+                                        orig_parent_dir = _os.path.dirname(original_path)
+                                        orig_parent_name = _os.path.basename(orig_parent_dir)
+                                        orig_filename_without_ext = _os.path.splitext(_os.path.basename(original_path))[0]
+
+                                        if orig_parent_name.lower() in image_extensions:
+                                            orig_grandparent_dir = _os.path.dirname(orig_parent_dir)
+                                            csv_relative = _os.path.join(
+                                                orig_grandparent_dir, "csv", orig_filename_without_ext + ".csv"
+                                            )
+                                        else:
+                                            csv_relative = original_path.replace(
+                                                _os.path.splitext(original_path)[1],
+                                                ".csv",
+                                            )
+
                                         csv_cwd = _os.path.join(
                                             _os.getcwd(), csv_relative
                                         )
@@ -835,10 +869,18 @@ def _handle_image_with_csv(
                 if hasattr(fig_obj, "export_as_csv_for_sigmaplot"):
                     sigmaplot_data = fig_obj.export_as_csv_for_sigmaplot()
                     if sigmaplot_data is not None and not sigmaplot_data.empty:
-                        # For CSV files, we want them in the same directory as the image
-                        csv_sigmaplot_path = spath.replace(
-                            ext_wo_dot, "csv"
-                        ).replace(".csv", "_for_sigmaplot.csv")
+                        # Determine SigmaPlot CSV path based on parent directory name
+                        if parent_name.lower() in image_extensions:
+                            grandparent_dir = _os.path.dirname(parent_dir)
+                            csv_dir = _os.path.join(grandparent_dir, "csv")
+                            csv_sigmaplot_path = _os.path.join(csv_dir, filename_without_ext + "_for_sigmaplot.csv")
+                        else:
+                            csv_sigmaplot_path = spath.replace(
+                                ext_wo_dot, "csv"
+                            ).replace(".csv", "_for_sigmaplot.csv")
+
+                        # Ensure parent directory exists
+                        _os.makedirs(_os.path.dirname(csv_sigmaplot_path), exist_ok=True)
                         # Save directly using _save to avoid path doubling
                         # Don't pass image-specific kwargs to CSV save
                         _save(
@@ -852,10 +894,21 @@ def _handle_image_with_csv(
 
                         # Create symlink_to for SigmaPlot CSV if it was specified for the image
                         if symlink_to:
-                            csv_sigmaplot_symlink_to = (
-                                _os.path.splitext(symlink_to)[0]
-                                + "_for_sigmaplot.csv"
-                            )
+                            symlink_parent_dir = _os.path.dirname(symlink_to)
+                            symlink_parent_name = _os.path.basename(symlink_parent_dir)
+                            symlink_filename_without_ext = _os.path.splitext(_os.path.basename(symlink_to))[0]
+
+                            if symlink_parent_name.lower() in image_extensions:
+                                symlink_grandparent_dir = _os.path.dirname(symlink_parent_dir)
+                                csv_sigmaplot_symlink_to = _os.path.join(
+                                    symlink_grandparent_dir, "csv", symlink_filename_without_ext + "_for_sigmaplot.csv"
+                                )
+                            else:
+                                csv_sigmaplot_symlink_to = (
+                                    _os.path.splitext(symlink_to)[0]
+                                    + "_for_sigmaplot.csv"
+                                )
+
                             _symlink_to(
                                 csv_sigmaplot_path,
                                 csv_sigmaplot_symlink_to,
@@ -881,11 +934,26 @@ def _handle_image_with_csv(
     # Save metadata as JSON if collected
     if collected_metadata is not None and not dry_run:
         try:
-            # Save JSON in same directory as image with .json extension
-            # Example: ./path/to/output/fig.png -> ./path/to/output/fig.json
-            json_path = _os.path.splitext(spath)[0] + ".json"
+            # Check if the path contains an image extension directory (e.g., ./png/, ./jpg/)
+            # If so, save JSON in a parallel ./json/ directory
+            # Example: ./path/to/output/png/fig.png -> ./path/to/output/json/fig.json
+            # Example: ./path/to/output/fig.png -> ./path/to/output/fig.json (same dir)
+            image_extensions = ['png', 'jpg', 'jpeg', 'gif', 'tiff', 'tif', 'svg', 'pdf']
+            parent_dir = _os.path.dirname(spath)
+            parent_name = _os.path.basename(parent_dir)
+            filename_without_ext = _os.path.splitext(_os.path.basename(spath))[0]
+
+            if parent_name.lower() in image_extensions:
+                # Parent directory is named after an image extension (e.g., png/)
+                # Create parallel json/ directory
+                grandparent_dir = _os.path.dirname(parent_dir)
+                json_dir = _os.path.join(grandparent_dir, "json")
+                json_path = _os.path.join(json_dir, filename_without_ext + ".json")
+            else:
+                # Save JSON in same directory as image
+                json_path = _os.path.splitext(spath)[0] + ".json"
 
-            # Ensure parent directory exists (should already exist from image save)
+            # Ensure parent directory exists
             _os.makedirs(_os.path.dirname(json_path), exist_ok=True)
 
             # Save metadata as JSON
@@ -900,7 +968,19 @@ def _handle_image_with_csv(
 
             # Create symlink_to for JSON if it was specified for the image
             if symlink_to:
-                json_symlink_to = _os.path.splitext(symlink_to)[0] + ".json"
+                # Apply same directory transformation for symlink
+                symlink_parent_dir = _os.path.dirname(symlink_to)
+                symlink_parent_name = _os.path.basename(symlink_parent_dir)
+                symlink_filename_without_ext = _os.path.splitext(_os.path.basename(symlink_to))[0]
+
+                if symlink_parent_name.lower() in image_extensions:
+                    symlink_grandparent_dir = _os.path.dirname(symlink_parent_dir)
+                    json_symlink_to = _os.path.join(
+                        symlink_grandparent_dir, "json", symlink_filename_without_ext + ".json"
+                    )
+                else:
+                    json_symlink_to = _os.path.splitext(symlink_to)[0] + ".json"
+
                 _symlink_to(json_path, json_symlink_to, True)
 
             # Create symlink for JSON manually if needed
@@ -915,11 +995,22 @@ def _handle_image_with_csv(
                     if "specified_path" in frame.frame.f_locals:
                         original_path = frame.frame.f_locals["specified_path"]
                         if isinstance(original_path, str):
-                            # Replace extension to get JSON path structure
-                            json_relative = original_path.replace(
-                                _os.path.splitext(original_path)[1],
-                                ".json",
-                            )
+                            # Apply same directory transformation for symlink
+                            orig_parent_dir = _os.path.dirname(original_path)
+                            orig_parent_name = _os.path.basename(orig_parent_dir)
+                            orig_filename_without_ext = _os.path.splitext(_os.path.basename(original_path))[0]
+
+                            if orig_parent_name.lower() in image_extensions:
+                                orig_grandparent_dir = _os.path.dirname(orig_parent_dir)
+                                json_relative = _os.path.join(
+                                    orig_grandparent_dir, "json", orig_filename_without_ext + ".json"
+                                )
+                            else:
+                                json_relative = original_path.replace(
+                                    _os.path.splitext(original_path)[1],
+                                    ".json",
+                                )
+
                             json_cwd = _os.path.join(_os.getcwd(), json_relative)
                             _symlink(json_path, json_cwd, True, True)
                             break

scitex/io/_save_modules/_image.py

@@ -1,12 +1,11 @@
 #!/usr/bin/env python3
 # -*- coding: utf-8 -*-
-# Timestamp: "2025-05-18 14:52:34 (ywatanabe)"
-# File: /ssh:sp:/home/ywatanabe/proj/scitex_repo/src/scitex/io/_save_modules/_save_image.py
-# ----------------------------------------
+# Timestamp: "2025-12-01 12:23:32 (ywatanabe)"
+# File: /home/ywatanabe/proj/scitex-code/src/scitex/io/_save_modules/_image.py
+
 import os
+
 __FILE__ = __file__
-__DIR__ = os.path.dirname(__FILE__)
-# ----------------------------------------
 
 import io as _io
 import logging
@@ -17,15 +16,23 @@ from PIL import Image
 logger = logging.getLogger(__name__)
 
 
-def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-right', verbose=False, **kwargs):
+def save_image(
+    obj,
+    spath,
+    metadata=None,
+    add_qr=False,
+    qr_position="bottom-right",
+    verbose=False,
+    **kwargs,
+):
     # Add URL to metadata if not present
     if metadata is not None:
         if verbose:
             logger.info(f"📝 Saving figure with metadata to: {spath}")
 
-        if 'url' not in metadata:
+        if "url" not in metadata:
             metadata = dict(metadata)
-            metadata['url'] = 'https://scitex.ai'
+            metadata["url"] = "https://scitex.ai"
             if verbose:
                 logger.info("  • Auto-added URL: https://scitex.ai")
 
@@ -35,12 +42,16 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
                 logger.info(f"  • Adding QR code at position: {qr_position}")
             try:
                 from .._qr_utils import add_qr_to_figure
+
                 # Only add QR for matplotlib figures
-                if hasattr(obj, 'savefig') or (hasattr(obj, 'figure') and hasattr(obj.figure, 'savefig')):
-                    fig = obj if hasattr(obj, 'savefig') else obj.figure
+                if hasattr(obj, "savefig") or (
+                    hasattr(obj, "figure") and hasattr(obj.figure, "savefig")
+                ):
+                    fig = obj if hasattr(obj, "savefig") else obj.figure
                     obj = add_qr_to_figure(fig, metadata, position=qr_position)
             except Exception as e:
                 import warnings
+
                 warnings.warn(f"Failed to add QR code: {e}")
 
     # png
@@ -67,7 +78,7 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
         # matplotlib
         else:
             # Use kwargs dpi if provided, otherwise default to 300
-            save_kwargs = {'format': 'tiff', 'dpi': kwargs.get('dpi', 300)}
+            save_kwargs = {"format": "tiff", "dpi": kwargs.get("dpi", 300)}
             save_kwargs.update(kwargs)
             try:
                 obj.savefig(spath, **save_kwargs)
@@ -85,7 +96,9 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
             obj.write_image(buf, format="png")
             buf.seek(0)
             img = Image.open(buf)
-            img.convert("RGB").save(spath, "JPEG", quality=100, subsampling=0, optimize=False)
+            img.convert("RGB").save(
+                spath, "JPEG", quality=100, subsampling=0, optimize=False
+            )
             buf.close()
 
         # PIL image
@@ -95,7 +108,7 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
 
         # matplotlib
         else:
-            save_kwargs = {'format': 'png'}
+            save_kwargs = {"format": "png"}
             save_kwargs.update(kwargs)
             try:
                 obj.savefig(buf, **save_kwargs)
@@ -105,7 +118,9 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
             buf.seek(0)
             img = Image.open(buf)
             # Save JPEG with very high quality settings for daily use (quality=98 is near-lossless)
-            img.convert("RGB").save(spath, "JPEG", quality=100, subsampling=0, optimize=False)
+            img.convert("RGB").save(
+                spath, "JPEG", quality=100, subsampling=0, optimize=False
+            )
             buf.close()
         del obj
 
@@ -125,7 +140,7 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
         # matplotlib
         else:
             buf = _io.BytesIO()
-            save_kwargs = {'format': 'png'}
+            save_kwargs = {"format": "png"}
             save_kwargs.update(kwargs)
             try:
                 obj.savefig(buf, **save_kwargs)
@@ -144,7 +159,7 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
             obj.write_image(file=spath, format="svg")
         # Matplotlib
         else:
-            save_kwargs = {'format': 'svg'}
+            save_kwargs = {"format": "svg"}
             save_kwargs.update(kwargs)
             try:
                 obj.savefig(spath, **save_kwargs)
@@ -160,15 +175,15 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
         # PIL Image - convert to PDF
         elif isinstance(obj, Image.Image):
             # Convert RGBA to RGB if needed
-            if obj.mode == 'RGBA':
-                rgb_img = Image.new('RGB', obj.size, (255, 255, 255))
+            if obj.mode == "RGBA":
+                rgb_img = Image.new("RGB", obj.size, (255, 255, 255))
                 rgb_img.paste(obj, mask=obj.split()[3])
                 rgb_img.save(spath, "PDF")
             else:
                 obj.save(spath, "PDF")
         # Matplotlib
         else:
-            save_kwargs = {'format': 'pdf'}
+            save_kwargs = {"format": "pdf"}
             save_kwargs.update(kwargs)
             try:
                 obj.savefig(spath, **save_kwargs)
@@ -179,12 +194,14 @@ def save_image(obj, spath, metadata=None, add_qr=False, qr_position='bottom-righ
     # Embed metadata if provided
     if metadata is not None:
         from .._metadata import embed_metadata
+
         try:
             embed_metadata(spath, metadata)
             if verbose:
-                logger.info(f"  • Embedded metadata: {metadata}")
+                logger.debug(f"  • Embedded metadata: {metadata}")
         except Exception as e:
             import warnings
+
             warnings.warn(f"Failed to embed metadata: {e}")
 
 # EOF