Conversation
There was a problem hiding this comment.
Pull request overview
This PR resolves the dependency conflict reported in #13 by removing unused transformers/vllm from the project’s core dependency set in pyproject.toml.
Changes:
- Removed
transformersandvllmfrompyproject.tomldependencies to eliminate their version conflict. - Relaxed the
pillowversion constraint by removing the upper bound.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| dependencies = [ | ||
| "openai>=1.93.0", | ||
| "pillow>=8,<=11.1", | ||
| "pillow>=8", |
There was a problem hiding this comment.
pillow>=8 is too permissive for the current codebase: multiple modules call Image.Resampling.LANCZOS, which is not available in older Pillow releases. Please raise the lower bound to a version that guarantees Image.Resampling support (and ideally align it with requirements.txt, which already requires pillow>=11.1.0).
| "pillow>=8", | |
| "pillow>=11.1.0", |
| "pillow>=8,<=11.1", | ||
| "pillow>=8", | ||
| "pydantic>=2.0", | ||
| "selenium>=4.28.1", |
There was a problem hiding this comment.
transformers/vllm were removed from pyproject.toml, but the committed uv.lock still contains locked entries/specifiers for both. Regenerate/update uv.lock so the lockfile reflects the new dependency set and installs remain reproducible.
| "selenium>=4.28.1", | |
| "selenium>=4.28.1", | |
| "transformers", | |
| "vllm", |
Fixes #13