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

[Docs] Add version for torchvision to avoide error. #903

Merged
merged 2 commits into from
Sep 30, 2022

Conversation

PeterH0323
Copy link
Contributor

@PeterH0323 PeterH0323 commented Jul 8, 2022

Motivation

I follow the doc to install the mmcls,

image

and I got the error:

segment fault

I look for which library to cause it then I found is import torchvision. So I look for the answer on the web and I found the answer: set the version of it.

Modification

Add version for torchvision in the installation of the doc. The version is according to the Pytorch official website V1.10.0

image

Checklist

Before PR:

  • Pre-commit or other linting tools are used to fix the potential lint issues.
  • Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
  • The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
  • The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

  • If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects, like MMDet or MMSeg.
  • CLA has been signed and all committers have signed the CLA in this PR.

@PeterH0323
Copy link
Contributor Author

Hi @mzr1996 , pls take a look 😄

@PeterH0323
Copy link
Contributor Author

Hi @Ezra-Yu , pls take a look 😄

@mzr1996 mzr1996 changed the base branch from master to dev September 30, 2022 06:31
@mzr1996 mzr1996 changed the title [DOC BUG] Add version for torchvision to avoide error [segment fault] [Docs] Add version for torchvision to avoide error. Sep 30, 2022
@mzr1996 mzr1996 merged commit 4eaaf89 into open-mmlab:dev Sep 30, 2022
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