On Tue, Oct 12, 2010 at 10:55 AM, Keith Goodman <kwgood...@gmail.com> wrote:
> Thanks for the code. I added matplotlib's yahoo historical quote code
> to my labeled array package, la. While doing so I noticed a couple of
> possible bugs:
>
> - The doc string for parse_yahoo_historical says that the volume is
> adjusted. But I don't see a line of code that does the adjustment.


We don't do the adjustment, Yahoo does.  Eg, take a look at CROX
around their 6/15/07 split.

  http://finance.yahoo.com/q/hp?s=CROX&a=05&b=1&c=2007&d=06&e=1&f=2007&g=d

IDC reports consolidated volume on 6/14/07 (pre-split) was 4,366,319
shares.  Yahoo reports 8,726,000 -- which is close to 2x the raw
volume (I assume the difference is in how some of the shares
transacted on non-primary exchanges are counted).  Likewise,  on
6/13/07, IDC reports
7,852,268 and Yahoo reports 15,544,200, which is close to 2x.  So it
appears they are backward split adjusting the volume.

> quotes_historical_yahoo builds an error message from a variable
> named "url" which doesn't exist in the scope of the function

Fixed -- this was a legacy error message when we used to pass in urls
but now pass in filehandles

> "import time" and "from matplotlib.cbook import is_string_like" are not used

Fixed.

Thanks for the report -- fixed on the branch (8742) and will be merged
to the trunk when MD resuscitates svnmerge....

JDH

------------------------------------------------------------------------------
Beautiful is writing same markup. Internet Explorer 9 supports
standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
Spend less time writing and  rewriting code and more time creating great
experiences on the web. Be a part of the beta today.
http://p.sf.net/sfu/beautyoftheweb
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Reply via email to