refactor: Consolidate Python requirements to pyproject.toml #4245
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Merged all dependencies from container/deps/requirements*.txt into pyproject.toml - Added dependency groups for test, standard, and deploy dependencies - Updated Dockerfiles and Earthfile to use pyproject.toml instead of requirements files - This centralizes all Python dependency management in one place 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
The container build was failing because the main pyproject.toml depends on ai-dynamo-runtime==0.7.0, but this package wasn't available during the dependency installation phase. The issue was in the build order: 1. Main package dependencies were being installed first 2. Runtime wheel was built but not yet installed 3. Dependency resolution failed for ai-dynamo-runtime==0.7.0 Fixed by: 1. Installing ai-dynamo-runtime wheel first from wheel_builder stage 2. Then installing main package dependencies from pyproject.toml 3. This ensures runtime is available when resolving main dependencies This addresses the pyproject.toml consolidation work and enables successful container builds. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Background
This PR consolidates Python requirements management by moving all dependencies to
pyproject.toml, eliminating the need for separate requirements files. During this consolidation, a container build issue was discovered and fixed.Changes
Requirements Consolidation
Container Build Fix
Test Results
✅ Container Environment: Successfully tested with VLLM framework
✅ Unit Tests: All tests pass (5/5 including mypy checks)
✅ GPU Integration: Confirmed CUDA availability and functionality
✅ Build Process: Container build works with consolidated requirements
Benefits
🤖 Generated with Claude Code