scitex 2.13.0__py3-none-any.whl → 2.15.1__py3-none-any.whl

scitex/bridge/_figrecipe.py

@@ -56,7 +56,7 @@ def save_with_recipe(
     dpi : int
         Resolution for image output.
     **kwargs
-        Additional arguments passed to savefig.
+        Additional arguments passed to savefig (including facecolor).
 
     Returns
     -------
@@ -76,12 +76,9 @@ def save_with_recipe(
         storage = get_storage(path)
         storage.ensure_exists()
 
-        # Get underlying matplotlib figure
-        mpl_fig = fig._fig_mpl if hasattr(fig, "_fig_mpl") else fig
-
-        # 1. Save image
+        # 1. Save image - use fig.savefig() to get facecolor fix from FigWrapper
         image_path = storage.path / "plot.png"
-        mpl_fig.savefig(image_path, dpi=dpi, **kwargs)
+        _save_figure_image(fig, image_path, dpi=dpi, **kwargs)
         result["image"] = image_path
 
         # 2. Save SigmaPlot CSV
@@ -105,8 +102,7 @@ def save_with_recipe(
 
     else:
         # Single file save (image + sidecars)
-        mpl_fig = fig._fig_mpl if hasattr(fig, "_fig_mpl") else fig
-        mpl_fig.savefig(path, dpi=dpi, **kwargs)
+        _save_figure_image(fig, path, dpi=dpi, **kwargs)
         result["image"] = path
 
         # Save CSV sidecar
@@ -131,6 +127,42 @@ def save_with_recipe(
     return result
 
 
+def _save_figure_image(fig, path: Path, dpi: int = 300, **kwargs):
+    """Save figure image using the best available method with facecolor support.
+
+    Uses fig.savefig() when available (FigWrapper or RecordingFigure) to get
+    the facecolor override fix for transparent figures.
+    """
+    # Check if this is a figrecipe RecordingFigure - use fr.save() for full support
+    if FIGRECIPE_AVAILABLE:
+        try:
+            from figrecipe._wrappers import RecordingFigure
+
+            if isinstance(fig, RecordingFigure):
+                # Use figrecipe's save with facecolor support
+                facecolor = kwargs.pop("facecolor", None)
+                fr.save(
+                    fig,
+                    path,
+                    save_recipe=False,  # Recipe saved separately
+                    dpi=dpi,
+                    facecolor=facecolor,
+                    verbose=False,
+                    **kwargs,
+                )
+                return
+        except (ImportError, AttributeError):
+            pass
+
+    # Use fig.savefig() if available (FigWrapper has facecolor fix)
+    if hasattr(fig, "savefig"):
+        fig.savefig(path, dpi=dpi, **kwargs)
+    else:
+        # Fallback to matplotlib figure's savefig
+        mpl_fig = fig._fig_mpl if hasattr(fig, "_fig_mpl") else fig
+        mpl_fig.savefig(path, dpi=dpi, **kwargs)
+
+
 def _save_recipe_to_path(
     fig,
     path: Path,

scitex/bridge/_stats_plt.py

@@ -1,5 +1,4 @@
 #!/usr/bin/env python3
-# -*- coding: utf-8 -*-
 # File: ./src/scitex/bridge/_stats_plt.py
 # Time-stamp: "2024-12-09 10:00:00 (ywatanabe)"
 """
@@ -25,11 +24,9 @@ When explicit x, y are provided, they use whatever coordinate system
 the caller intends (typically data coordinates unless transform is set).
 """
 
-from typing import Optional, Dict, Any, Union, List
-import warnings
+from typing import Any, List, Optional
 
 # Import GUI classes from FTS (single source of truth)
-from scitex.io.bundle._stats import Position
 
 # StatResult is now a dict - the GUI-specific StatResult is deprecated
 StatResult = dict
@@ -214,6 +211,7 @@ def _parse_stat_annotation(text: str) -> Optional[StatResult]:
     def _create_stat_dict(test_type, statistic_name, statistic_value, p_value):
         """Create a simple stat result dict."""
         from scitex.stats.utils import p2stars
+
         return {
             "test_type": test_type,
             "test_category": "other",

scitex/bridge/_stats_vis.py

@@ -1,5 +1,4 @@
 #!/usr/bin/env python3
-# -*- coding: utf-8 -*-
 # File: ./src/scitex/bridge/_stats_vis.py
 # Time-stamp: "2024-12-09 10:00:00 (ywatanabe)"
 """
@@ -24,14 +23,15 @@ This differs from _stats_plt which uses axes coordinates (0-1 normalized).
 When bridging between plt and vis, coordinate transformation may be needed.
 """
 
-from typing import Optional, Dict, Any, List, Tuple
+from typing import Dict, List, Optional, Tuple
 
 # Import GUI classes from FTS (single source of truth)
-from scitex.io.bundle._stats import Position, StatPositioning
+from scitex.io.bundle.kinds._stats import Position
 
 # Legacy model imports - may not be available
 try:
-    from scitex.canvas.model import AnnotationModel, FigureModel, AxesModel, TextStyle
+    from scitex.canvas.model import AnnotationModel, AxesModel, FigureModel, TextStyle
+
     VIS_MODEL_AVAILABLE = True
 except ImportError:
     AnnotationModel = None
@@ -219,7 +219,9 @@ def position_stat_annotation(
     y = y_min + base_y * y_range
 
     # Check overlap and adjust if needed
-    min_dist = stat_result.positioning.min_distance_mm if stat_result.positioning else 2.0
+    min_dist = (
+        stat_result.positioning.min_distance_mm if stat_result.positioning else 2.0
+    )
 
     for ex_x, ex_y in existing:
         dist = ((x - ex_x) ** 2 + (y - ex_y) ** 2) ** 0.5

scitex/browser/collaboration/__init__.py

@@ -18,7 +18,6 @@ Features:
 Version: 0.1.0-alpha (experimental)
 """
 
-__version__ = "0.1.0-alpha"
 __experimental__ = True
 
 # Import components

scitex/browser/core/ChromeProfileManager.py

@@ -1,9 +1,9 @@
 #!/usr/bin/env python3
-# -*- coding: utf-8 -*-
 # Timestamp: "2025-10-11 07:53:19 (ywatanabe)"
 # File: /home/ywatanabe/proj/scitex_repo/src/scitex/browser/core/ChromeProfileManager.py
 # ----------------------------------------
 from __future__ import annotations
+
 import os
 
 __FILE__ = "./src/scitex/browser/core/ChromeProfileManager.py"
@@ -73,7 +73,7 @@ class ChromeProfileManager:
         extensions_path = profile_dir / "Default" / "Extensions"
 
         if not extensions_path.exists():
-            return {key: False for key in self.EXTENSIONS}
+            return dict.fromkeys(self.EXTENSIONS, False)
 
         for key, ext_info in self.EXTENSIONS.items():
             ext_id = ext_info["id"]
@@ -268,7 +268,8 @@ class ChromeProfileManager:
         Args:
             source_profile_name: Name of source profile (default: "system")
 
-        Returns:
+        Returns
+        -------
             True if sync succeeded, False otherwise
         """
         import time
@@ -343,6 +344,12 @@ class ChromeProfileManager:
             return True
 
         except subprocess.CalledProcessError as e:
+            # Exit code 23 = partial transfer (often just timestamp issues on WSL)
+            # Check if it's only "failed to set times" errors - these are harmless
+            if e.returncode == 23 and "failed to set times" in e.stderr:
+                # Files copied successfully, just couldn't preserve timestamps
+                logger.debug("Profile sync complete (timestamps not preserved)")
+                return True
             logger.error(f"rsync failed (exit code {e.returncode}): {e.stderr}")
             return False
         except FileNotFoundError:

scitex/browser/debugging/_browser_logger.py

@@ -1,9 +1,9 @@
 #!/usr/bin/env python3
-# -*- coding: utf-8 -*-
 # Timestamp: "2025-12-09 (ywatanabe)"
 # File: /home/ywatanabe/proj/scitex-code/src/scitex/browser/debugging/_browser_logger.py
 # ----------------------------------------
 from __future__ import annotations
+
 import os
 
 __FILE__ = "./src/scitex/browser/debugging/_browser_logger.py"
@@ -87,7 +87,8 @@ async def log_page_async(
         verbose: Enable/disable visual popups and screenshots (default True)
         level: Log level - one of: debug, info, success, warning, error, fail (default "info")
 
-    Returns:
+    Returns
+    -------
         bool: True if successful, False otherwise
 
     Example:
@@ -115,9 +116,8 @@ async def log_page_async(
         When verbose=False, only logger messages are generated without
         any visual feedback or screenshots, making it suitable for production.
     """
-    # Log to terminal
-    log_func = getattr(logger, level, logger.info)
-    log_func(f"    {func_name} - {message}")
+    # Log to terminal at DEBUG level (visual popups still show at specified level)
+    logger.debug(f"    {func_name} - {message}")
 
     # Check if this log level would actually be shown
     level_numeric = {
@@ -371,9 +371,8 @@ async def log_page_async(
                     path=str(screenshot_full_path),
                     full_page=False,
                 )
-                log_func(
+                logger.debug(
                     f"    {func_name} - Screenshot: {screenshot_full_path}",
-                    c="grey",
                 )
             except Exception as e:
                 log_func(
@@ -599,8 +598,6 @@ def main(args):
 
 def parse_args() -> argparse.Namespace:
     """Parse command line arguments."""
-    import scitex as stx
-
     parser = argparse.ArgumentParser(
         description="Popup and screenshot capture utility for debugging"
     )

scitex/canvas/README.md

@@ -1,7 +1,7 @@
 <!-- ---
 !-- Timestamp: 2025-12-08
 !-- Author: ywatanabe
-!-- File: /home/ywatanabe/proj/scitex-code/src/scitex/vis/README.md
+!-- File: /home/ywatanabe/proj/scitex-python/src/scitex/vis/README.md
 !-- --- -->
 
 # scitex.vis - Canvas-Based Figure Composition

scitex/canvas/_mcp/__init__.py

@@ -0,0 +1,4 @@
+#!/usr/bin/env python3
+# File: __init__.py
+"""MCP server components."""
+

scitex/canvas/_mcp/handlers.py

@@ -0,0 +1,372 @@
+#!/usr/bin/env python3
+# Timestamp: 2026-01-08
+# File: src/scitex/canvas/_mcp.handlers.py
+# ----------------------------------------
+
+"""
+MCP Handler implementations for SciTeX canvas module.
+
+Provides async handlers for multi-panel figure composition.
+"""
+
+from __future__ import annotations
+
+import asyncio
+from typing import Optional
+
+
+async def create_canvas_handler(
+    parent_dir: str,
+    canvas_name: str,
+    width_mm: float = 180,
+    height_mm: float = 120,
+) -> dict:
+    """
+    Create a new canvas workspace.
+
+    Parameters
+    ----------
+    parent_dir : str
+        Parent directory for canvas
+    canvas_name : str
+        Name for the canvas
+    width_mm : float
+        Canvas width in mm
+    height_mm : float
+        Canvas height in mm
+
+    Returns
+    -------
+    dict
+        Success status and canvas info
+    """
+    try:
+        import scitex.canvas as canvas_module
+
+        loop = asyncio.get_event_loop()
+        await loop.run_in_executor(
+            None,
+            lambda: canvas_module.create_canvas(
+                parent_dir,
+                canvas_name,
+            ),
+        )
+
+        canvas_path = canvas_module.get_canvas_path(parent_dir, canvas_name)
+
+        return {
+            "success": True,
+            "canvas_name": canvas_name,
+            "canvas_path": str(canvas_path),
+            "size_mm": {"width": width_mm, "height": height_mm},
+            "message": f"Created canvas '{canvas_name}' at {canvas_path}",
+        }
+    except Exception as e:
+        return {
+            "success": False,
+            "error": str(e),
+        }
+
+
+async def add_panel_handler(
+    parent_dir: str,
+    canvas_name: str,
+    panel_name: str,
+    source: str,
+    x_mm: float = 0,
+    y_mm: float = 0,
+    width_mm: float = 50,
+    height_mm: float = 50,
+    label: Optional[str] = None,
+) -> dict:
+    """
+    Add a panel to a canvas.
+
+    Parameters
+    ----------
+    parent_dir : str
+        Parent directory containing canvas
+    canvas_name : str
+        Canvas name
+    panel_name : str
+        Name for this panel
+    source : str
+        Path to source file
+    x_mm, y_mm : float
+        Position in mm
+    width_mm, height_mm : float
+        Size in mm
+    label : str, optional
+        Panel label (A, B, C...)
+
+    Returns
+    -------
+    dict
+        Success status and panel info
+    """
+    try:
+        import scitex.canvas as canvas_module
+
+        loop = asyncio.get_event_loop()
+        await loop.run_in_executor(
+            None,
+            lambda: canvas_module.add_panel(
+                parent_dir=parent_dir,
+                canvas_name=canvas_name,
+                panel_name=panel_name,
+                source=source,
+                xy_mm=(x_mm, y_mm),
+                size_mm=(width_mm, height_mm),
+                label=label or "",
+            ),
+        )
+
+        return {
+            "success": True,
+            "canvas_name": canvas_name,
+            "panel_name": panel_name,
+            "source": source,
+            "position_mm": {"x": x_mm, "y": y_mm},
+            "size_mm": {"width": width_mm, "height": height_mm},
+            "label": label,
+            "message": f"Added panel '{panel_name}' to canvas '{canvas_name}'",
+        }
+    except Exception as e:
+        return {
+            "success": False,
+            "error": str(e),
+        }
+
+
+async def list_panels_handler(
+    parent_dir: str,
+    canvas_name: str,
+) -> dict:
+    """
+    List all panels in a canvas.
+
+    Parameters
+    ----------
+    parent_dir : str
+        Parent directory
+    canvas_name : str
+        Canvas name
+
+    Returns
+    -------
+    dict
+        Success status and panel list
+    """
+    try:
+        import scitex.canvas as canvas_module
+
+        loop = asyncio.get_event_loop()
+        panels = await loop.run_in_executor(
+            None,
+            lambda: canvas_module.list_panels(parent_dir, canvas_name),
+        )
+
+        return {
+            "success": True,
+            "canvas_name": canvas_name,
+            "count": len(panels) if panels else 0,
+            "panels": panels or [],
+        }
+    except Exception as e:
+        return {
+            "success": False,
+            "error": str(e),
+        }
+
+
+async def remove_panel_handler(
+    parent_dir: str,
+    canvas_name: str,
+    panel_name: str,
+) -> dict:
+    """
+    Remove a panel from a canvas.
+
+    Parameters
+    ----------
+    parent_dir : str
+        Parent directory
+    canvas_name : str
+        Canvas name
+    panel_name : str
+        Panel to remove
+
+    Returns
+    -------
+    dict
+        Success status
+    """
+    try:
+        import scitex.canvas as canvas_module
+
+        loop = asyncio.get_event_loop()
+        await loop.run_in_executor(
+            None,
+            lambda: canvas_module.remove_panel(parent_dir, canvas_name, panel_name),
+        )
+
+        return {
+            "success": True,
+            "canvas_name": canvas_name,
+            "panel_name": panel_name,
+            "message": f"Removed panel '{panel_name}' from canvas '{canvas_name}'",
+        }
+    except Exception as e:
+        return {
+            "success": False,
+            "error": str(e),
+        }
+
+
+async def export_canvas_handler(
+    parent_dir: str,
+    canvas_name: str,
+    output_path: Optional[str] = None,
+    format: Optional[str] = None,
+    dpi: int = 300,
+) -> dict:
+    """
+    Export canvas to file.
+
+    Parameters
+    ----------
+    parent_dir : str
+        Parent directory
+    canvas_name : str
+        Canvas name
+    output_path : str, optional
+        Output file path
+    format : str, optional
+        Output format (png, pdf, svg)
+    dpi : int
+        Output DPI
+
+    Returns
+    -------
+    dict
+        Success status and output path
+    """
+    try:
+        from pathlib import Path
+
+        import scitex.canvas as canvas_module
+
+        # Build output path if not provided
+        if output_path is None:
+            canvas_path = canvas_module.get_canvas_path(parent_dir, canvas_name)
+            fmt = format or "png"
+            output_path = str(Path(canvas_path) / "exports" / f"{canvas_name}.{fmt}")
+
+        loop = asyncio.get_event_loop()
+        result_path = await loop.run_in_executor(
+            None,
+            lambda: canvas_module.export_canvas(
+                parent_dir,
+                canvas_name,
+                output_path,
+            ),
+        )
+
+        return {
+            "success": True,
+            "canvas_name": canvas_name,
+            "output_path": str(result_path) if result_path else output_path,
+            "format": format or Path(output_path).suffix.lstrip("."),
+            "dpi": dpi,
+            "message": f"Exported canvas to {output_path}",
+        }
+    except Exception as e:
+        return {
+            "success": False,
+            "error": str(e),
+        }
+
+
+async def list_canvases_handler(parent_dir: str) -> dict:
+    """
+    List all canvases in a directory.
+
+    Parameters
+    ----------
+    parent_dir : str
+        Directory to search
+
+    Returns
+    -------
+    dict
+        Success status and canvas list
+    """
+    try:
+        import scitex.canvas as canvas_module
+
+        loop = asyncio.get_event_loop()
+        canvases = await loop.run_in_executor(
+            None,
+            lambda: canvas_module.list_canvases(parent_dir),
+        )
+
+        return {
+            "success": True,
+            "parent_dir": parent_dir,
+            "count": len(canvases) if canvases else 0,
+            "canvases": canvases or [],
+        }
+    except Exception as e:
+        return {
+            "success": False,
+            "error": str(e),
+        }
+
+
+async def canvas_exists_handler(
+    parent_dir: str,
+    canvas_name: str,
+) -> dict:
+    """
+    Check if a canvas exists.
+
+    Parameters
+    ----------
+    parent_dir : str
+        Parent directory
+    canvas_name : str
+        Canvas name
+
+    Returns
+    -------
+    dict
+        Success status and existence flag
+    """
+    try:
+        import scitex.canvas as canvas_module
+
+        exists = canvas_module.canvas_exists(parent_dir, canvas_name)
+
+        return {
+            "success": True,
+            "canvas_name": canvas_name,
+            "exists": exists,
+        }
+    except Exception as e:
+        return {
+            "success": False,
+            "error": str(e),
+        }
+
+
+__all__ = [
+    "create_canvas_handler",
+    "add_panel_handler",
+    "list_panels_handler",
+    "remove_panel_handler",
+    "export_canvas_handler",
+    "list_canvases_handler",
+    "canvas_exists_handler",
+]
+
+# EOF