Docker cli optimization #62
Conversation
- Add optimized Dockerfile with multi-stage build - Create requirements-docker.txt with essential dependencies only - Update pyproject.toml with latest dependency versions - Add docker-compose.yml for easy deployment - Create build and push scripts for automation - Add comprehensive Docker documentation (DOCKER.md) - Update README.md with Docker installation option - Improve .gitignore for Docker and development files Co-authored-by: openhands <[email protected]>
- Remove unnecessary port exposure and health checks from Dockerfile - Simplify docker-compose.yml for CLI tool (remove Redis, ports, health checks) - Update DOCKER.md with CLI-focused usage examples - Fix README.md Docker example to show proper CLI usage - Add more API key environment variables (Cohere, Mistral, Google) Co-authored-by: openhands <[email protected]>
- Add immutabledict, jsonlines, html2text, tree-sitter packages - Add faiss-cpu, evaluate, tiktoken, mpmath for ML functionality - Add backoff, tenacity, overrides, psutil for utility functions - Add segtok, rank-bm25 for text processing - Ensure all 74 core tests pass in Docker environment - Support complete CLI functionality in containerized environment Co-authored-by: openhands <[email protected]>
h-albert-lee
requested review from
baeseongsu and
hw-oh
and removed request for
hw-oh
|
일단 빌드는 되는데, fail하는 케이스들 조금 더 분석만 해보겠습니다! 별다른 이슈 없으면 dockerhub에 upload해서 링크도 공유드릴게요 |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
hw-oh
force-pushed
the
docker-cli-optimization
branch
from
b9dde4c to
3fd6df7
Compare
📋 Summary
Optimized Docker configuration for CLI-based usage of the haerae-evaluation-toolkit, adding missing packages.
🔧 Changes Made
Docker Configuration:
Dependencies:
Added 19 missing packages: immutabledict, jsonlines, html2text, tree-sitter, faiss-cpu, evaluate, tiktoken, mpmath, backoff, tenacity, overrides, psutil, segtok, rank-bm25, etc.
Updated requirements-docker.txt for complete functionality
Documentation:
Updated DOCKER.md for CLI usage patterns
Fixed README.md Docker examples to show CLI execution
✅ Verification Results
✅ Docker build successful (11.2GB)
✅ 74/117 tests passing (63% success rate) -> fail한 케이스는 대부분 api token이 없었음. dependency문제는 x
✅ Core CLI functionality working
✅ All major ML dependencies importing correctly
✅ docker-compose execution confirmed
🚀 Usage
Build and run
docker build -t haerae-evaluation-toolkit .