Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions ovmobilebench/builders/openvino.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
"""OpenVINO build system."""

import logging
import os
from pathlib import Path

from ovmobilebench.config.schema import OpenVINOConfig
Expand Down Expand Up @@ -63,6 +64,7 @@ def _configure_cmake(self):
str(self.build_dir),
"-GNinja",
f"-DCMAKE_BUILD_TYPE={self.config.build_type}",
f"-DOUTPUT_ROOT={os.getcwd()}/{self.build_dir}",
]

# Android-specific configuration
Expand Down Expand Up @@ -122,8 +124,8 @@ def _build(self):
def get_artifacts(self) -> dict[str, Path]:
"""Get paths to build artifacts."""
artifacts = {
"benchmark_app": self.build_dir / "bin" / "arm64-v8a" / "benchmark_app",
"libs": self.build_dir / "bin" / "arm64-v8a",
"benchmark_app": self.build_dir / "bin" / "aarch64" / "Release" / "benchmark_app",
"libs": self.build_dir / "bin" / "aarch64" / "Release",
}

# Verify artifacts exist
Expand Down
4 changes: 3 additions & 1 deletion ovmobilebench/pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,9 @@ def deploy(self) -> None:
logger.info("[DRY RUN] Would deploy to devices")
return

bundle_path = self.artifacts_dir / "packages" / f"ovbundle_{self.config.project.run_id}"
bundle_path = (
self.artifacts_dir / "packages" / f"ovbundle_{self.config.project.run_id}.tar.gz"
)

for serial in self.config.device.serials:
logger.info(f"Deploying to device: {serial}")
Expand Down
2 changes: 1 addition & 1 deletion ovmobilebench/runners/benchmark.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ def run_single(
}

if rc != 0:
logger.error(f"Benchmark failed: {stderr}")
logger.error(f"Benchmark failed:\n{stdout}\n{stderr}")

return result

Expand Down
4 changes: 2 additions & 2 deletions tests/builders/test_builders_openvino.py
Original file line number Diff line number Diff line change
Expand Up @@ -286,8 +286,8 @@ def test_get_artifacts_success(self, mock_ensure_dir, build_config):
artifacts = builder.get_artifacts()

expected = {
"benchmark_app": Path("/build/dir/bin/arm64-v8a/benchmark_app"),
"libs": Path("/build/dir/bin/arm64-v8a"),
"benchmark_app": Path("/build/dir/bin/aarch64/Release/benchmark_app"),
"libs": Path("/build/dir/bin/aarch64/Release"),
}
assert artifacts == expected

Expand Down
2 changes: 1 addition & 1 deletion tests/runners/test_runners_benchmark.py
Original file line number Diff line number Diff line change
Expand Up @@ -301,7 +301,7 @@ def test_run_single_logs_error_on_failure(self, mock_device, run_config, benchma
# Check that error was logged
mock_logger.error.assert_called_once()
error_call = mock_logger.error.call_args[0][0]
assert "Benchmark failed: benchmark failed" in error_call
assert "Benchmark failed:\n\nbenchmark failed" in error_call

def test_run_matrix_logs_progress(self, mock_device, run_config):
"""Test that run_matrix logs progress information."""
Expand Down
Loading