Curator icon indicating copy to clipboard operation
Curator copied to clipboard

ci: Unbound transformers in default dep

Open thomasdhc opened this issue 1 month ago • 1 comments

Description

Unbound transformers default dependency version. Move specific bounds to video.

Usage

# Add snippet demonstrating usage

Checklist

  • [ ] I am familiar with the Contributing Guide.
  • [ ] New or Existing tests cover these changes.
  • [ ] The documentation is up to date with these changes.

thomasdhc avatar Nov 21 '25 15:11 thomasdhc

Greptile Overview

Greptile Summary

Removed the pinned version constraint (==4.55.2) from the default transformers dependency, allowing flexibility while maintaining a minimum version (>=4.55.2) via the override-dependencies configuration.

  • Default dependencies now use unbounded transformers instead of ==4.55.2
  • Override-dependencies section enforces transformers>=4.55.2 as the minimum version
  • Lock file (uv.lock) updated to reflect both changes
  • PR description mentions moving specific bounds to video dependencies, but no transformers constraint was added to video_cpu or video_cuda12 sections despite video models (InternVideo2, Cosmos) using transformers

Confidence Score: 3/5

  • This PR is moderately safe to merge with some consideration needed
  • The dependency changes are technically sound (unbounded with minimum constraint via override), but the PR description mentions "Move specific bounds to video" which wasn't implemented. Video models use transformers but no video-specific constraint was added. The override ensures >=4.55.2 globally, which provides safety, but the incomplete implementation per the description reduces confidence.
  • Pay attention to pyproject.toml - verify whether video dependencies need explicit transformers>=4.55.2 constraint as mentioned in PR description

Important Files Changed

File Analysis

Filename Score Overview
pyproject.toml 3/5 Unbounded transformers in default dependencies while keeping >=4.55.2 constraint in overrides; PR description mentions moving bounds to video but no video-specific constraint was added
uv.lock 5/5 Lock file updated to reflect the unbounded transformers dependency and >=4.55.2 override constraint

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant ProjFile as pyproject.toml
    participant Override as override-dependencies
    participant Lock as uv.lock
    participant Pkg as Package Manager
    
    Dev->>ProjFile: Remove version pin (==4.55.2)
    Dev->>ProjFile: Set unbounded transformers
    Dev->>Override: Update to >=4.55.2 constraint
    ProjFile->>Lock: Trigger lock file update
    Lock->>Lock: Update transformers specifier
    Note over ProjFile,Override: Default deps: unbounded<br/>Override: >=4.55.2 minimum
    Pkg->>Override: Apply override constraint
    Pkg->>Pkg: Install transformers >=4.55.2
    Note over Pkg: All environments get<br/>transformers >=4.55.2

greptile-apps[bot] avatar Nov 21 '25 15:11 greptile-apps[bot]