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 installation requirements error #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

osbm
Copy link

@osbm osbm commented May 24, 2024

Fixes #4

Main problem with using only setup.py is that if you have installation requirements running setup.py on a fresh environment is going to cause error because you are trying to import those dependencies before calling setup function.

With the addition of pyproject.toml file you can install those dependencies before running setup.py file. So you can install the package with only running

pip install .

or

pip install git+https://github.com/Dao-AILab/fast-hadamard-transform

If you want to try before merging you can just run this on a fresh environment:

pip install git+https://github.com/osbm/fast-hadamard-transform

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.

Failure during install - "packaging" not found
1 participant