Skip to content

Conversation

@IlyasMoutawwakil
Copy link
Member

No description provided.

@IlyasMoutawwakil IlyasMoutawwakil added the cpu [CI] Requires and enables running all CPU tests label Aug 17, 2025
@IlyasMoutawwakil IlyasMoutawwakil added cuda [CI] Requires and enables running all CUDA tests misc [CI] Requires and enables running all basic utility tests across multiple platforms and removed cpu [CI] Requires and enables running all CPU tests cuda [CI] Requires and enables running all CUDA tests labels Aug 17, 2025
@IlyasMoutawwakil IlyasMoutawwakil added energy_star and removed misc [CI] Requires and enables running all basic utility tests across multiple platforms labels Aug 17, 2025
@IlyasMoutawwakil IlyasMoutawwakil added cuda [CI] Requires and enables running all CUDA tests and removed energy_star cuda [CI] Requires and enables running all CUDA tests labels Aug 17, 2025
@IlyasMoutawwakil IlyasMoutawwakil added cpu [CI] Requires and enables running all CPU tests energy_star misc [CI] Requires and enables running all basic utility tests across multiple platforms and removed cuda [CI] Requires and enables running all CUDA tests cpu [CI] Requires and enables running all CPU tests energy_star misc [CI] Requires and enables running all basic utility tests across multiple platforms labels Aug 19, 2025
@IlyasMoutawwakil IlyasMoutawwakil added the vllm [CI] Requires and enables running all vLLM tests label Aug 19, 2025
@IlyasMoutawwakil IlyasMoutawwakil force-pushed the uv branch 2 times, most recently from 0de2ace to 0f21594 Compare August 19, 2025 17:37

This comment was marked as outdated.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enables UV package manager as the primary dependency management tool for optimum-benchmark, replacing the traditional setup.py approach with modern pyproject.toml configuration.

  • Migration from setup.py to pyproject.toml with hatchling build backend
  • Addition of UV configuration files (uv.toml, .python-version)
  • Complete overhaul of CI/CD workflows to use UV instead of pip

Reviewed Changes

Copilot reviewed 32 out of 33 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
setup.py Removed legacy setup.py configuration file
pyproject.toml Added modern project configuration with dependencies and build system
uv.toml Added UV-specific configuration for package management
.python-version Set Python 3.10 as the project version
tests/conftest.py Added pytest configuration to handle empty test scenarios
Makefile Comprehensive rewrite with UV-based commands and improved organization
README.md Updated installation instructions to recommend UV
.github/workflows/* Updated all CI workflows to use UV instead of pip
optimum_benchmark/trackers/energy.py Added null check for emission data
optimum_benchmark/task_utils.py Fixed diffusers import compatibility issue
optimum_benchmark/preprocessors/dataset_preprocessor.py Simplified audio preprocessing logic
optimum_benchmark/backends/vllm/config.py Removed redundant configuration
optimum_benchmark/backends/vllm/backend.py Fixed vLLM API compatibility issues
optimum_benchmark/backends/pytorch/backend.py Enhanced torch compile support
optimum_benchmark/backends/ipex/backend.py Added OS compatibility fix
examples/cuda_vllm_llama.yaml Added vLLM version environment variable

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@IlyasMoutawwakil IlyasMoutawwakil removed the vllm [CI] Requires and enables running all vLLM tests label Aug 19, 2025
@IlyasMoutawwakil IlyasMoutawwakil merged commit 50c063b into main Aug 19, 2025
40 of 42 checks passed
@IlyasMoutawwakil IlyasMoutawwakil deleted the uv branch August 19, 2025 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants