-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Correct run instructions in Readme #87
Conversation
Release 0.0.45
Fix this for future users so they can get it up and running without having to probe the GitHub forum
Hey, Paul! Thanks for the contribution. Could you explain why I just tested here on a new conda environment and it worked. |
@ogabrielluiz I did not set up a conda environment. I just did the pip install so there is no cli command named If the idea case is to make a conda environment please add that to the readme. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, use the code block and add both commands and this should be good to go.
The conda environment is used to isolate the installation to make the test more reliable. |
FWIW the current instructions work for me, just
|
You can pick any Python interpreter to install and then to run, for example:
|
@ogabrielluiz What do you mean by this comment? do you want me to add a line to make a conda environment? I would think a comment would be more helpful or having 2 options like: Next, run:
or
|
Yes. Having both options displayed looks good to me. No conda environment though. That was used to test the error you found. |
Fix this for future users so they can get it up and running without having to probe the GitHub forum