Skip to content

Bump torchvision to 0.21.0 #1312

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

menicosia
Copy link

When I tried this example on a fresh install (OSX Python, deployed by Brew) I got the following error:

% pip install -r requirements.txt
ERROR: Ignored the following yanked versions: 0.1.6, 0.1.7, 0.1.8, 0.1.9, 0.2.0, 0.2.1, 0.2.2, 0.2.2.post2, 0.2.2.post3
ERROR: Could not find a version that satisfies the requirement torchvision==0.20.0 (from versions: 0.21.0)
ERROR: No matching distribution found for torchvision==0.20.0

When I bumped it to v0.21.0, the error went away.

I think this is a harmless PR. If I'm mistaken, please reject.

When I tried this example on a fresh install (OSX Python, deployed by Brew) I got the following error:

```
% pip install -r requirements.txt
ERROR: Ignored the following yanked versions: 0.1.6, 0.1.7, 0.1.8, 0.1.9, 0.2.0, 0.2.1, 0.2.2, 0.2.2.post2, 0.2.2.post3
ERROR: Could not find a version that satisfies the requirement torchvision==0.20.0 (from versions: 0.21.0)
ERROR: No matching distribution found for torchvision==0.20.0
```

When I bumped it to v0.21.0, the error went away.
Copy link

netlify bot commented Feb 14, 2025

Deploy Preview for pytorch-examples-preview canceled.

Name Link
🔨 Latest commit 6959d81
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-examples-preview/deploys/67aeb4fa4da26d000832ca3c

@@ -1,2 +1,2 @@
torch
torchvision==0.20.0
torchvision==0.21.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these examples should work on the latest torch and torchvision packages unless they deliberately test the deprecated API and soon will be dropped. As such we need to drop version pin entirely instead. That was done in the recently merged #1334 for this example. I suggest to close this PR. Otherwise, please, join discussion in issue #1329.

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.

3 participants