Re: [matplotlib-devel] Potential Bug in axes.py

[Eric F. <ef...@ha...>]

Brad Chivari wrote:
> SUBJECT:
> Filtering out 0 bar height/width not working
> 
> FILE:
> matplotlib/ trunk/ matplotlib/ lib/ matplotlib/ axes.py
> 
> PROBLEM:
> xmin = np.amin(width[width!=0]) # filter out the 0 width rects
> ymin = np.amin(height[height!=0]) # filter out the 0 height rects
> 
> These aren't using proper python list comprehension and don't work as expected (for me anyway).
> 
> SOLUTION:
> Shouldn't they be something like:
> xmin = np.amin([w for w in width if w != 0])
> ymin = np.amin([h for h in height if h != 0])
> 
> Once I changed them they seem to work properly.

I have applied a slight modification to your fix, which seems to me to 
be the right approach, given that the bar implementation is based on 
lists rather than ndarrays.

Thanks.

Eric