-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
pandas.io.data.Options is broken due to Yahoo options page change #8612
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
cc @dstephens99 well, someone needs to have a look. @strimp099 if you would have a look would be gr8!. |
Guess that's a call to action :) will take a look. JAS On Oct 23, 2014, at 12:42, jreback [email protected] wrote: cc @dstephens99 well, someone needs to have a look. @strimp099 if you would have a look would be gr8!. — |
I'm seeing this too:
|
This is the error you'll see. The XML paths have changed. JAS On Oct 23, 2014, at 14:02, Polly Powledge [email protected] wrote: I'm seeing this too:
|
I'm working on it. Its changed quite a bit. |
I forked the repo and made a few updates if you want to take a look. |
It's a lot easier to handle it to be honest |
can I help with it? I parsed the new site 2 weeks ago. |
Hi, thanks for taking a look at this. I'm afraid I'm still seeing the problem, though. Is there a way I could check to see that I'm running with the correct bits, besides checking the version?
|
Maybe this could help. I think there is problem with going to unix timestamp from old method. |
I'm new to pandas but took a run at this:
But it's not clear to me if it really should be returning expirations unique for each month and year as I'm doing here (and which the method name implies), or if I should be returning expirations, period. If it's the latter then I'll change the code. Can anyone enlighten me? Alas, this is not the only thing the changed Yahoo page breaks: _get_option_data() is broken too. I haven't looked at that yet. Any other suggestions? I know I'll need to clean up the xpath string. It's just begging to have another form added to the page and break things all over again. UPDATE: As I'm going through the Options class, I see that it's just using the expiry month and year throughout the code. If you're not using dates, how this would work for months with multiple expiration dates, e.g., weeklies and quarterlies? Are the non-standard options getting skipped over? Or am I missing something here? |
Yahoo used to have all the expiries for a month on the same page. My PR (#8631) changes the code so that it uses a specific date now. |
The example in the docs (http://pandas-docs.github.io/pandas-docs-travis/remote_data.html#yahoo-finance-options) is broken, so this certainly needs to be addressed for 0.15.1 (fixed or temporarily removed from the docs) |
#8631 Will fix the docs. |
Look at accepted answer fixed. Does not use pandas...but Unix date is fixed. Url fixed. can be run in loop |
Anything i am missing here?Still not working i guess
|
@chiragmatkar what version are you using? Working for me with 0.16.2 |
@jorisvandenbossche 0.14.1 |
@jorisvandenbossche Thanks .upgraded and its working fine now. |
Partially addresses pandas-devgh-8254. Closes pandas-devgh-8612 because pd.to_datetime has a format arg. [ci skip]
Yahoo reconfigured their options page (http://finance.yahoo.com/q/op?s=AAPL+Options) which broke the entire Options class.
Are there plans for a fix in a future release?
The text was updated successfully, but these errors were encountered: