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

Change requirements #396

Merged
merged 10 commits into from
Jul 4, 2022
Merged

Change requirements #396

merged 10 commits into from
Jul 4, 2022

Conversation

ashwinvaidya17
Copy link
Collaborator

Description

  • Torchvision 0.13.0 ends up installing torch 0.12.0 which leads to segmentation fault in OpenVINO (2022.1.0) inferencer. This PR fixes the versions.

@ashwinvaidya17
Copy link
Collaborator Author

Closing as this change will be merged with visualizer pr

@ashwinvaidya17
Copy link
Collaborator Author

Reopening as this adds range to the requirements

@ashwinvaidya17 ashwinvaidya17 reopened this Jul 1, 2022
torchmetrics>=0.9.0
torchvision>=0.9.1
torchtext>=0.9.1
torchmetrics>=0.9.0,<=0.9.1
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need to restrict torchmetrics? It's a different library by the Lightning team,

@samet-akcay samet-akcay merged commit c5220c0 into development Jul 4, 2022
@samet-akcay samet-akcay deleted the ashwin/change_requirements branch July 4, 2022 12:43
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