Re: [matplotlib-devel] request for code review: updates to Sankey class

2012-10-16 Thread Kevin Davies
Thanks for your comments! see below... On 10/16/2012 10:14 AM, Damon McDougall wrote: > On Tue, Oct 16, 2012 at 2:55 PM, Kevin Davies wrote: >> Hello, >> >> I made a few minor changes to the Sankey class. They are listed at: >> https://github.com/kdavies4/matplotlib/compare/master...sankey5 >> >

Re: [matplotlib-devel] request for code review: updates to Sankey class

2012-10-16 Thread Damon McDougall
On Tue, Oct 16, 2012 at 2:55 PM, Kevin Davies wrote: > Hello, > > I made a few minor changes to the Sankey class. They are listed at: > https://github.com/kdavies4/matplotlib/compare/master...sankey5 > > Please review this and let me know if I can submit a pull request. > > Thanks. > > Kevin Tha

[matplotlib-devel] request for code review: updates to Sankey class

2012-10-16 Thread Kevin Davies
Hello, I made a few minor changes to the Sankey class.  They are listed at: https://github.com/kdavies4/matplotlib/compare/master...sankey5 Please review this and let me know if I can submit a pull request. Thanks. Kevin -