This looks good to me. Could you add a test the produces one or more pie charts which uses the frame and a non zero center?

Others may have different opinions but I think this is a new feature so it should go on the master branch not the 1.4.x branch, but since the changes are rather uninvolved I might be convinced otherwise.

I am also in favor of this targeting the master branch. I am hoping to keep 1.4.x bug fixes only and do more frequent point releases.

Don't worry about re-targetting it, we can just do the merge by into master when it is ready.

Added tests for three non zero centered pie charts with axis frame on. (And
pep8 conformance)

On 29 August 2014 18:33, Thomas A Caswell [email protected] wrote:

I am also in favor of this targeting the master branch. I am hoping to
keep 1.4.x bug fixes only and do more frequent point releases.

Don't worry about re-targetting it, we can just do the merge by into
master when it is ready.

—
Reply to this email directly or view it on GitHub
#3433 (comment)
.

Kimmo Palin

Merged to master in e7717f7