Skip to content
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

pin torchvision version to 0.16.2 in dev-requirements #698

Closed
wants to merge 1 commit into from

Conversation

JKSenthil
Copy link
Contributor

Summary:

Context

We are seeing failed unit test, caused by torchvision error
{F1452783880}
Torchvision recently updated to 0.17, which is when I suspect this issue occurred.

This Diff

For now, pinning the version to the last 0.16.x one, as there were no failing unit tests before

Differential Revision: D53456248

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D53456248

JKSenthil added a commit to JKSenthil/tnt that referenced this pull request Feb 6, 2024
Summary:

# Context
We are seeing failed unit test, caused by torchvision error
{F1452783880}
Torchvision recently updated to 0.17, which is when I suspect this issue occurred. 
# This Diff
For now, pinning the version to the last 0.16.x one, as there were no failing unit tests before

Reviewed By: gunchu

Differential Revision: D53456248
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D53456248

Summary:

# Context
We are seeing failed unit test, caused by torchvision error
{F1452783880}
Torchvision recently updated to 0.17, which is when I suspect this issue occurred. 
# This Diff
For now, pinning the version to the last 0.16.x one, as there were no failing unit tests before

Reviewed By: gunchu

Differential Revision: D53456248
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D53456248

@JKSenthil JKSenthil force-pushed the export-D53456248 branch 2 times, most recently from b5dcb4d to 17805a5 Compare February 6, 2024 01:19
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