Bug in PatchCollection

Development matplotlib-devel
1

Hi,

I just hit a bug in PatchCollection, at line 891 in collections.py:

         if match_original:
             def determine_facecolor(patch):
                 if patch.fill():
                     return patch.get_facecolor()
                 return [0, 0, 0, 0]

The problem is that patch.fill is a boolean attribute, not a function. (Or at least it is for polygons and circles.) You only hit this if you override the default and specify match_original=True.

Ryan

···

--
Ryan May
Graduate Research Assistant
School of Meteorology
University of Oklahoma

2

Ryan May wrote:

Hi,

I just hit a bug in PatchCollection, at line 891 in collections.py:

         if match_original:
             def determine_facecolor(patch):
                 if patch.fill():
                     return patch.get_facecolor()
                 return [0, 0, 0, 0]

The problem is that patch.fill is a boolean attribute, not a function. (Or at least it is for polygons and circles.) You only hit this if you override the default and specify match_original=True.

Ryan

Fixed in 5764. Thanks, Ryan.

Eric