Replaced fairseq with fairseq-fixed for compatibility with modern Python versions - #46
Replaced fairseq with fairseq-fixed for compatibility with modern Python versions#46Adithya-Sakaray wants to merge 1 commit into
Conversation
Updated fairseq to fairseq-fixed in dependencies. This fixed the issue in modern python interpreters.
There was a problem hiding this comment.
Pull request overview
This PR updates the dependency from fairseq to fairseq-fixed to enable compatibility with modern Python versions. The original fairseq package has installation issues on recent Python interpreters, and fairseq-fixed is a maintained fork that preserves the same API while resolving these installation problems.
Changes:
- Replaced fairseq with fairseq-fixed in app/dependencies.txt
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| tensorboardX | ||
| pyarrow | ||
| fairseq | ||
| fairseq-fixed |
There was a problem hiding this comment.
Replacing the official fairseq dependency with fairseq-fixed introduces an unpinned third-party package that will be automatically fetched and executed in your environments, creating a significant supply-chain risk. If the fairseq-fixed package or its publishing account is ever compromised, attackers could ship malicious code that runs with your application’s privileges and access to secrets or data. To reduce this risk, ensure fairseq-fixed is from a trusted, reviewed source and pin it to a vetted, immutable version (or vendor it) rather than relying on the latest mutable release.
This PR updates the dependency from fairseq to fairseq-fixed.
The original fairseq package fails to install on modern Python interpreters due to outdated build and dependency constraints. fairseq-fixed is a drop-in replacement that preserves the same API while restoring installability on current Python versions.
Why this change
What changed
If maintainers prefer an alternative approach, I am happy to adjust.