windows support (#19)

* update SSH testing: improve CI environment compatibility with mock fallback, enhance macOS SSH configuration, and add robust connection retries

* update SSH testing: improve CI environment compatibility  (#18)

* update SSH testing: improve CI environment compatibility with mock fallback, enhance macOS SSH configuration, and add robust connection retries

* update tests and workflows: make SSH username dynamic in tests and add concurrency control to GitHub Actions

* update workflows: simplify device serial description in bench workflow inputs

* update README: revise supported platforms table with expanded host-device compatibility details

* update README: revise supported platforms table with expanded host-device compatibility details

* refactor tests and core utilities: use `datetime.now(timezone.utc)` for consistent timestamping, replace `subprocess.Popen` with `subprocess.run` for simpler command execution, enhance filesystem utility robustness, and improve cross-platform path handling

* refactor tests and core utilities: remove unnecessary whitespace, simplify `Mock` return values in subprocess tests, standardize string quotation style, and update README with additional platform compatibility

* update SSH setup and testing: add Windows support with PowerShell script, refactor setup script generation for platform detection, and improve CI workflow integration

* refactor tests: remove unnecessary whitespace in SSH-related test files for improved readability and standardization

* refactor SSH setup: simplify script for CI environments by removing OpenSSH Server installation and configuration logic, streamline SSH connection testing, and improve logging for compatibility checks

* enhance CLI and CI encoding handling: add UTF-8 defaults for Windows, improve interactive and CI pipeline behavior, and enforce strict error handling in SSH setup and device tests

* remove unnecessary whitespace: clean up extra blank lines and trailing spaces in `cli.py` for improved readability and code style consistency

* handle exceptions explicitly: replace bare `except` with `except Exception` in `cli.py` for clearer and safer error handling

* refactor SSH tests and setup: simplify platform checks, enhance logging for failed SSH connections, streamline macOS SSH daemon startup logic, and add detailed diagnostics for CI environments

* refactor SSH tests and setup: simplify platform checks, enhance logging for failed SSH connections, streamline macOS SSH daemon startup logic, and add detailed diagnostics for CI environments

* remove redundant platform checks: simplify `test_ssh_in_ci` by removing unused platform-specific logic

* update SSH testing: remove unused variables in test scripts for cleanup and better maintainability

* add shell specification for SSH commands in CI workflow for consistency and compatibility

* install and configure OpenSSH Server: enhance CI SSH setup with server installation, configuration updates, improved key handling, and robust connection testing

* clean up `test_ssh_in_ci`: adjust formatting for consistent style and readability

* update SSH configurations and tests: replace `localhost` with `127.0.0.1` for consistency, remove unused experiment files, and enhance `test_ssh_in_ci` with Paramiko test server integration

* validate SSH device config: ensure `host` is specified in device configuration and raise `ConfigError` if missing; minor imports adjustment

* format SSH code and tests: adjust indentation, replace single quotes with double quotes for consistency, and apply minor cleanup for improved readability

* add platform-specific logic to SSH tests: update `test_main_setup` and `test_main_all` to handle Windows and Unix separately, ensuring platform compatibility in script generation

* refactor SSH tests: adjust formatting in `test_generate_ssh_config` for consistent style and readability

* remove SSH localhost experiments and enhance mock mode: delete unused `ssh_localhost_ci.yaml`, refactor SSH device logic with `mock_mode` for dry-runs, and simplify SSH CI testing with Paramiko test server integration

* remove generate_ssh_config.py and associated files: delete unused script, configuration, and test files related to SSH config generation for CI

Author: allnes

.github/codecov.yml

@@ -4,8 +4,6 @@ ignore:
   - "**/__pycache__"
   - "**/*.pyc"
   - "setup.py"
-  - "scripts/generate_ssh_config.py"
-  - "scripts/test_ssh_device_ci.py"
 
 coverage:
   status:

.github/workflows/bench.yml

@@ -20,7 +20,7 @@ jobs:
   ci-matrix:
     strategy:
       matrix:
-        os: [ubuntu-latest, macos-latest] # , windows-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
     uses: ./.github/workflows/ci-orchestrator.yml
     with:
       os: ${{ matrix.os }}

.github/workflows/stage-device-tests.yml

@@ -28,24 +28,20 @@ jobs:
           pip install -r requirements.txt
           pip install -e .
 
-      - name: Set up SSH server
+      - name: Test SSH with Paramiko test server
+        shell: bash
         run: |
-          python scripts/generate_ssh_config.py --type setup
-          bash scripts/setup_ssh_ci.sh || echo "SSH setup had warnings, continuing..."
-
-      - name: List SSH devices
-        run: |
-          ovmobilebench list-ssh-devices || echo "Command not yet implemented"
-
-      - name: Test SSH deployment
-        run: |
-          python scripts/generate_ssh_config.py --type test
-          python scripts/test_ssh_device_ci.py
+          set -e
+          python tests/test_ssh_device_ci.py
 
       - name: Run benchmark dry-run via SSH
+        shell: bash
+        env:
+          PYTHONIOENCODING: utf-8
+          PYTHONUTF8: 1
         run: |
-          python scripts/generate_ssh_config.py --type config
-          ovmobilebench all -c experiments/ssh_localhost_ci.yaml --dry-run || true
+          set -e
+          ovmobilebench all -c experiments/ssh_test_ci.yaml --dry-run
 
       - name: Upload SSH test results
         if: always()

.gitignore

@@ -131,7 +131,5 @@ dmypy.json
 .claude
 CLAUDE.md
 
-# Generated CI configs
-experiments/ssh_localhost_ci.yaml
+# Test results
 experiments/results/
-scripts/setup_ssh_ci.sh

README.md

@@ -56,6 +56,7 @@ cat experiments/results/*.csv
 | Windows | x86_64       | Android   | ARM64       | ADB       | adbutils  | ✅ Stable  |
 | Linux   | x86_64       | Linux     | ARM64/ARM32 | SSH       | paramiko  | ✅ Stable  |
 | macOS   | x86_64/ARM64 | Linux     | ARM64/ARM32 | SSH       | paramiko  | ✅ Stable  |
+| Windows | x86_64       | Linux     | ARM64/ARM32 | SSH       | paramiko  | ✅ Stable  |
 | Any     | Any          | iOS       | ARM64       | USB       | -         | 🚧 Planned |
 
 ## 📋 Requirements

docs/ci-cd.md

@@ -613,7 +613,7 @@ def collect_metrics(results_path, metadata):
 def send_to_influxdb(metrics):
     from influxdb import InfluxDBClient
     
-    client = InfluxDBClient('localhost', 8086, database='ovmobilebench')
+    client = InfluxDBClient('127.0.0.1', 8086, database='ovmobilebench')
     
     points = []
     for result in metrics['results']:

experiments/ssh_localhost.yaml

@@ -0,0 +1,43 @@
+# SSH test configuration for CI
+project:
+  name: ssh-test-ci
+  run_id: ci-test
+
+# SSH device configuration for mock testing
+device:
+  type: linux_ssh
+  host: 127.0.0.1
+  username: testuser
+  push_dir: /tmp/ovmobilebench
+
+# Build configuration (disabled for CI)
+build:
+  enabled: false
+  openvino_repo: /tmp/openvino
+
+# Dummy models for testing
+models:
+  - name: dummy_model
+    path: /tmp/dummy_model.xml
+    precision: FP32
+
+# Run configuration
+run:
+  repeats: 1
+  warmup: 0
+  cooldown_sec: 0
+  matrix:
+    niter: [10]
+    nstreams: ["1"]
+    device: ["CPU"]
+
+# Reporting
+report:
+  sinks:
+    - type: csv
+      path: experiments/results/ssh_test.csv
+    - type: json
+      path: experiments/results/ssh_test.json
+  tags:
+    test_type: ssh_test_ci
+    ci: true

experiments/ssh_test_ci.yaml

@@ -3,6 +3,8 @@
 # Apply typer compatibility patch
 from ovmobilebench import typer_patch  # noqa: F401
 
+import os
+import sys
 import typer
 from pathlib import Path
 from typing import Optional
@@ -12,14 +14,27 @@
 from ovmobilebench.config.loader import load_experiment
 from ovmobilebench.pipeline import Pipeline
 
+# Set UTF-8 encoding for Windows
+if sys.platform == "win32":
+    os.environ["PYTHONIOENCODING"] = "utf-8"
+    # Also set console code page to UTF-8 if possible
+    try:
+        import subprocess
+
+        subprocess.run("chcp 65001", shell=True, capture_output=True)
+    except Exception:
+        pass
+
 app = typer.Typer(
     name="ovmobilebench",
     help="End-to-end benchmarking pipeline for OpenVINO on mobile devices",
     add_completion=False,
     pretty_exceptions_enable=False,  # Disable pretty exceptions
     rich_markup_mode=None,  # Disable Rich formatting
 )
-console = Console()
+
+# Configure console with safe encoding for Windows
+console = Console(legacy_windows=True if sys.platform == "win32" else None)
 
 
 @app.command()
@@ -33,7 +48,7 @@ def build(
     cfg = load_experiment(config)
     pipeline = Pipeline(cfg, verbose=verbose, dry_run=dry_run)
     pipeline.build()
-    console.print("[bold green]✓ Build completed[/bold green]")
+    console.print("[bold green][OK] Build completed[/bold green]")
 
 
 @app.command()
@@ -47,7 +62,7 @@ def package(
     cfg = load_experiment(config)
     pipeline = Pipeline(cfg, verbose=verbose, dry_run=dry_run)
     pipeline.package()
-    console.print("[bold green]✓ Package created[/bold green]")
+    console.print("[bold green][OK] Package created[/bold green]")
 
 
 @app.command()
@@ -61,7 +76,7 @@ def deploy(
     cfg = load_experiment(config)
     pipeline = Pipeline(cfg, verbose=verbose, dry_run=dry_run)
     pipeline.deploy()
-    console.print("[bold green]✓ Deployment completed[/bold green]")
+    console.print("[bold green][OK] Deployment completed[/bold green]")
 
 
 @app.command()
@@ -77,7 +92,7 @@ def run(
     cfg = load_experiment(config)
     pipeline = Pipeline(cfg, verbose=verbose, dry_run=dry_run)
     pipeline.run(timeout=timeout, cooldown=cooldown)
-    console.print("[bold green]✓ Benchmarks completed[/bold green]")
+    console.print("[bold green][OK] Benchmarks completed[/bold green]")
 
 
 @app.command()
@@ -90,7 +105,7 @@ def report(
     cfg = load_experiment(config)
     pipeline = Pipeline(cfg, verbose=verbose)
     pipeline.report()
-    console.print("[bold green]✓ Reports generated[/bold green]")
+    console.print("[bold green][OK] Reports generated[/bold green]")
 
 
 @app.command()
@@ -102,11 +117,10 @@ def all(
     cooldown: Optional[int] = typer.Option(None, "--cooldown", help="Cooldown between runs"),
 ):
     """Execute complete pipeline: build, package, deploy, run, and report."""
-    with Progress(
-        SpinnerColumn(),
-        TextColumn("[progress.description]{task.description}"),
-        console=console,
-    ) as progress:
+    # Check if we're in CI environment
+    is_ci = os.environ.get("CI", "").lower() == "true"
+
+    try:
         cfg = load_experiment(config)
         pipeline = Pipeline(cfg, verbose=verbose, dry_run=dry_run)
 
@@ -118,16 +132,42 @@ def all(
             ("Generating reports...", pipeline.report),
         ]
 
-        for description, stage_func in stages:
-            task = progress.add_task(description, total=None)
-            try:
-                stage_func()
-                progress.update(task, completed=True)
-            except Exception as e:
-                console.print(f"[bold red]✗ {description} failed: {e}[/bold red]")
-                raise
-
-    console.print("[bold green]✓ Pipeline completed successfully[/bold green]")
+        if is_ci or verbose:
+            # Simple output for CI or verbose mode
+            for description, stage_func in stages:
+                print(f"[*] {description}")
+                try:
+                    stage_func()
+                    print(f"[OK] {description} completed")
+                except Exception as e:
+                    print(f"[FAIL] {description} failed: {e}")
+                    raise
+            print("[OK] Pipeline completed successfully")
+        else:
+            # Rich progress bar for interactive use
+            spinner = SpinnerColumn(spinner_name="dots" if sys.platform == "win32" else "aesthetic")
+
+            with Progress(
+                spinner,
+                TextColumn("[progress.description]{task.description}"),
+                console=console,
+                transient=True,  # Clear progress when done
+            ) as progress:
+                for description, stage_func in stages:
+                    task = progress.add_task(description, total=None)
+                    try:
+                        stage_func()
+                        progress.update(task, completed=True)
+                    except Exception as e:
+                        console.print(f"[bold red][FAIL] {description} failed: {e}[/bold red]")
+                        raise
+
+            console.print("[bold green][OK] Pipeline completed successfully[/bold green]")
+    except UnicodeEncodeError as e:
+        # Fallback for encoding errors
+        print(f"Encoding error: {e}")
+        print("Pipeline failed due to encoding issues. Try setting PYTHONIOENCODING=utf-8")
+        sys.exit(1)
 
 
 @app.command()

ovmobilebench/cli.py

@@ -9,7 +9,7 @@
 def load_yaml(path: Path) -> Dict[str, Any]:
     """Load YAML configuration file."""
     if not path.exists():
-        raise FileNotFoundError(f"Configuration file not found: {path}")
+        raise FileNotFoundError(f"Configuration file not found: {path.as_posix()}")
 
     with open(path, "r") as f:
         data: Dict[str, Any] = yaml.safe_load(f)