-
Notifications
You must be signed in to change notification settings - Fork 42
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
update dependencies #14
Comments
Shall I take this one up? Updating the versions and verifying that the tests work right? |
i ran the tests against the latest packages in my virtual env. they are passing |
That's most of it. The manifest also has a declaration of the dependencies and needs to be updated. Also, there are no unit tests for pulling data from yahoo. So, those have to be checked manually, or else we need some basic integration tests. The options puller won't work between roughly midnight and 8 am EST because yahoo is resetting the servers. |
can you close this issue now? It still shows up in the list of open issues |
That was intentional. We'll have the same issue on each new version of the package. |
Use most recent stable versions of Pandas, NumPy, matplotlib, and Pandas DataReader.
The text was updated successfully, but these errors were encountered: