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

Fix for Windows install #163

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

Ivan-267
Copy link
Contributor

With the changes from the PR, I was able to install this in Windows 10 64 bit and run inference on my CPU.

Prior to the changes, pip install commands would fail at:

  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error    

With the error:

site-packages\setuptools\_distutils\util.py", line 134, in convert_path
          raise ValueError("path '%s' cannot end with '/'" % pathname)
      ValueError: path './' cannot end with '/'

According to the error message, I have removed the / which fixed the issue for me.
Please feel free to retest that everything is still fine for Linux when considering the PR.

@qgallouedec
Copy link
Member

Thanks @Ivan-267 for contributing :)

@qgallouedec qgallouedec merged commit 0104772 into huggingface:main Apr 25, 2024
3 checks passed
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