-
Notifications
You must be signed in to change notification settings - Fork 679
followup on Yahoo DataReader issues #356
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
Comments
When I put Similar problem in the docs. |
@jreback Do you plan to publish 0.5.0 soon? Thanks |
yes if this can be resolved soon |
I cannot reproduce this above with current
|
Based on the URI, this appears to be the previous API - I believe you just need the latest updates.
… On Jul 4, 2017, at 2:44 PM, Gábor Lipták ***@***.***> wrote:
I cannot reproduce this above with current master (installed as pip install git+git://github.com/pydata/pandas-datareader.git):
➜ pip list | grep pandas-datareader
pandas-datareader (0.5.0)
➜ jupyter-console
Jupyter console 5.1.0
Python 2.7.12+ (default, Sep 17 2016, 12:08:02)
Type "copyright", "credits" or "license" for more information.
IPython 5.4.1 -- An enhanced Interactive Python.
? -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help -> Python's own help system.
object? -> Details about 'object', use 'object??' for extra details.
In [1]: import pandas_datareader as pdr
In [2]: pdr.get_data_yahoo('SPY').tail()
Out[2]:
Open High Low Close Adj Close \
Date
2017-06-27 243.039993 243.380005 241.309998 241.330002 241.330002
2017-06-28 242.500000 243.720001 242.229996 243.490005 243.490005
2017-06-29 243.660004 243.720001 239.960007 241.350006 241.350006
2017-06-30 242.279999 242.710007 241.580002 241.800003 241.800003
2017-07-03 242.880005 243.380005 242.210007 242.210007 242.210007
Volume
Date
2017-06-27 82247700
2017-06-28 70042600
2017-06-29 103933000
2017-06-30 86820700
2017-07-03 39147200
In [3]:
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
you have to run the test and see what is xfailed these r all related to dividends in yahoo |
Yahoo works inconsistently ... For example: https://github.com/pydata/pandas-datareader/blob/master/pandas_datareader/tests/yahoo/test_yahoo.py#L215 (SPLIT) returns 0.0 and 0.14285714 https://github.com/pydata/pandas-datareader/blob/master/pandas_datareader/tests/yahoo/test_yahoo.py#L112 returns 251 and 252 (missing and containing 2013-12-31) How would you see the tests updated re this above? Thanks https://travis-ci.org/gliptak/pandas-datareader/jobs/250111075 |
right so the XPASS ones you can prob just take the decorator off and add the the get_data_interval is consistenty failing though so should be fixed (not sure what is wrong) |
I think for the inconsistencies you can just make the test less strict, e.g. accept 251 or 252 (odd that yahoo is like that), just add a comment about this. |
My only guess is that we need to control for a tie zone in the unix stamps.
… On Jul 4, 2017, at 4:56 PM, Jeff Reback ***@***.***> wrote:
I think for the inconsistencies you can just make the test less strict, e.g. accept 251 or 252 (odd that yahoo is like that), just add a comment about this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@rgkimball https://github.com/pydata/pandas-datareader/blob/master/pandas_datareader/yahoo/daily.py#L100 As example values, self.start comes in as 2010-01-01 00:00:00 and self.end comes in as 2013-01-27 00:00:00 Maybe self.end to be modified to 2013-01-27 23:59:59? Just during conversion or earlier in the code? This doesn't introduce time zones (yet) ... |
if someone has a chance.....this (and #296) are remaining for 0.5.0 |
Yahoo get_components is also broken #238 |
@jreback Could current |
I think the remainer of the xfails should be addressed. Then can do the release. |
Yahoo has been deprecated. |
in #355 I xfailed a couple of tests as they were not pulling data. These should be investigated before the release.
cc @rgkimball
The text was updated successfully, but these errors were encountered: