FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: Fixed torch install issue in CI by ntkathole · Pull Request #5668 · feast-dev/feast · GitHub

fix: Fixed torch install issue in CI - #5668

Merged
ntkathole merged 1 commit into
feast-dev:masterfrom
ntkathole:fix_torch
Oct 15, 2025
Merged

fix: Fixed torch install issue in CI#5668
ntkathole merged 1 commit into
feast-dev:masterfrom
ntkathole:fix_torch

Conversation

Copy link
Copy Markdown
Member

What this PR does / why we need it:

The torch release pipeline occasionally releases wheel builds to their index (https://download.pytorch.org/whl/cpu) before the PyPI release is available. This causes issues in our CI pipeline where:

  1. pip install torch torchvision --index-url https://download.pytorch.org/whl/cpu --force-reinstall installs the latest version (e.g., 2.9.0) from the torch wheel index
  2. uv pip sync then tries to sync with our pinned requirements, but the new version isn't on PyPI yet.
  3. This creates a version mismatch, causing installation failures or test failures

This PR implements the two-step process with a single uv pip sync command that uses --extra-index-url.

Signed-off-by: ntkathole <nikhilkathole2683@gmail.com>
ntkathole merged commit 366e5a8 into feast-dev:master Oct 15, 2025
21 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL