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

Incomplete requirement list to run tests #15

Open
smartie2076 opened this issue Mar 22, 2021 · 3 comments
Open

Incomplete requirement list to run tests #15

smartie2076 opened this issue Mar 22, 2021 · 3 comments

Comments

@smartie2076
Copy link

Hi @christopherjenness I think it would be nice if there was a requirements list to go though to install everything needed for the test file. Here the is what I had to do to set that up on my system:

pip install -U scikit-learn to install sklearn
pip install pytest
pip install hdbscan or conda install -c conda-forge hdbscan

I actually also expected the test folder to provide an example of the code´s application, not the assertions, I would add an example.py for that (eg. using the code in the README).

@barblin
Copy link

barblin commented Mar 22, 2021

You should find all the install requirements in requirements.txt, which is located in the source folder

@smartie2076
Copy link
Author

smartie2076 commented Mar 22, 2021

I know, and I also should have installed those automatically by clicking on "Install requirements":

grafik

and python setup.py install also passed with Finished processing dependencies of DBCV==0.0.0

Still, when I tried running the tests, it resulted in
grafik

As well as the same for pytest and hdbscan.

@smartie2076
Copy link
Author

If you are sure that the installation works flawlessly, you can ignore this (I was using pycharm and Windows 10 Pro by the way). I think what would be interesting though would still be a file example.py.

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

No branches or pull requests

2 participants