-
Notifications
You must be signed in to change notification settings - Fork 851
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
Modified the README.md with updated installation instruction. #88
Conversation
@femtotrader, @ryankennedyio - Are you happy with the new README.md? |
|
||
# Installation and Usage | ||
Ubuntu is the recommended platform on which to install QSTrader, but it will also work on Windows or Mac OSX under the Anaconda distribution (). |
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.
Unnecessary parentheses maybe?
Only other thing I might suggest is some info on how to analyse/view the .pkl files that are generated? Otherwise, good good. Maybe some tips about generating and passing in the config file as a Click parameter, but don't think necessary at this stage, as anyone running it will be diving pretty deep into the code anyway. |
Yeah, I think for the time being I'll ignore the Click parameter discussion. It might overcomplicate it for some users at this stage. Hopefully once we package it up via PyPI then the installation will be far easier. |
|
||
``` | ||
source ~/venv/qstrader/bin/activate | ||
pip install -r ~/projects/qstrader/requirements.txt | ||
pip install git+git://github.com/mhallsmoore/qstrader.git |
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.
I think
$ pip install git+git://
should be
$ pip install git+https://
No description provided.