Skip to content

fix: add missing runtime dependencies for torch-runtime install#115

Open
micheljung wants to merge 1 commit into
OpenMOSS:mainfrom
micheljung:main
Open

fix: add missing runtime dependencies for torch-runtime install#115
micheljung wants to merge 1 commit into
OpenMOSS:mainfrom
micheljung:main

Conversation

@micheljung
Copy link
Copy Markdown

Add accelerate to torch-runtime extras (required for device_map="auto" inference, not just finetuning) and document the FFmpeg system dependency in the README (required by torchcodec for audio I/O).

Add accelerate to torch-runtime extras (required for device_map="auto"
inference, not just finetuning) and document the FFmpeg system dependency
in the README (required by torchcodec for audio I/O).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@YWMditto
Copy link
Copy Markdown
Collaborator

Looks good overall, and I think this is mergeable.

One small documentation follow-up before merging would be ideal: the FFmpeg note is currently under the “Using Conda” section, but it also applies to the uv install path because torchcodec is part of torch-runtime there too. Could you move the FFmpeg system dependency note to the shared “Environment Setup” area before the Conda/uv subsections?

It would also be nice to mirror the same note in README_zh.md so the Chinese README stays aligned.

The accelerate>=1.10.1 addition to torch-runtime looks reasonable to me.

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