vqpy icon indicating copy to clipboard operation
vqpy copied to clipboard
verified publisher icon vqpy

Fix setup requirements

Open whzzt opened this issue 2 years ago • 2 comments

Why this change?

Current requirements.txt cannot set up the environment from stretch. This PR fixes it.

What does this PR include?

  • Add more package dependencies.
  • Altered the implementation of yolox dependencies. Now we do not need to install yolox for running VQPy.

User API changes

How did I test the PR?

New dependencies included

  • I have added the dependencies in setup.py.

whzzt avatar Oct 11 '23 23:10 whzzt

It seems that without this PR we can still install the whole package with pip install ., but we have to install it again and again for updates in the backend code. This PR, at least, provides a way to use VQPy without installing it.

whzzt avatar Oct 12 '23 00:10 whzzt

It seems that the GitHub workflow fails on VQPy installation due to AssertionError: torch is required for pre-compiling ops, please install it first. Is that a problem we need to fix?

shanyu-sys avatar Nov 08 '23 18:11 shanyu-sys