Added ind and bw_method kwargs to KdePlot

[khgitting]

Adds ability to set the bandwidth used for the kde and the indices at which it is evaluated. The first is passed to the bw_method kwarg for scipy.stats.gaussian_kde (for scipy >= 0.11.0); the second is passed to gkde.evaluate. For the case when scipy < 0.11.0 but bw_method is specified, bw_method is ignored (not passed to scipy.stats.gaussian_kde), but a warning is issued to alert the user.

#4298

[jreback]

looking good....pls add a mention in v0.13.0.txt and doc/source.release.rst (in the 0.13 section)

[jreback]

great...looks mergable (as soon as we start the 0.13 ones)...@cpcloud?

[cpcloud]

can u make this self.assertEqual(ax.get_yscale(), 'log') so that if it ever fails we can see what the value of the left hand side is? thanks.

[khgitting]

I can totally do that.

I modeled this test on the test_kde test:

    @slow
    def test_kde(self):
        _skip_if_no_scipy()
        _check_plot_works(self.ts.plot, kind='kde')
        _check_plot_works(self.ts.plot, kind='density')
        ax = self.ts.plot(kind='kde', logy=True)
        self.assert_(ax.get_yscale() == 'log')

    @slow
    def test_kde_kwargs(self):
        _skip_if_no_scipy()
        from numpy import linspace
        _check_plot_works(self.ts.plot, kind='kde', bw_method=.5, ind=linspace(-100,100,20))
        _check_plot_works(self.ts.plot, kind='density', bw_method=.5, ind=linspace(-100,100,20))
        ax = self.ts.plot(kind='kde', logy=True, bw_method=.5, ind=linspace(-100,100,20))
        self.assert_(ax.get_yscale() == 'log')

Will change in both to be consistent, and to add the usefulness you describe to the already existing test as well.

[jreback]

@khgitting this looks good, can you rebase on master, then good to go?

[khgitting]

@jreback done.

[jreback]

@khgitting can you squash to 1 commit?

(like a rebase, but use 's' on the other commits)

[khgitting]

@jreback Rebased, squashing, on top of a fresh master from fetching upstream.

[jreback]

@khgitting thanks!