bug report and fix for bracket arrow (annotations)

In http://matplotlib.sourceforge.net/users/annotations_guide.html ,
about 1/3 of the way down, there is a little demonstrator for the
different arrowstyles ->, <-, ]-, etc.

Looking at the figure closely, there is no difference between the "-["
and "]-" styles.

The fix for this is in patches.py, around line 3276 (matplotlib
1.0.0). Comments in that routine clipped out for brevity.

   class BracketA(_Bracket):
        def __init__(self, widthA=1., lengthA=0.2, angleA=None):

            super(ArrowStyle.BracketA, self).__init__(None,True,
                     widthA=widthA, lengthA=lengthA, angleA=angleA )

If you flip the first two args of the __init__ call, that fixes it:

   class BracketA(_Bracket):
        def __init__(self, widthA=1., lengthA=0.2, angleA=None):

            super(ArrowStyle.BracketA, self).__init__(True,None,
############## flip these arguments
                     widthA=widthA, lengthA=lengthA, angleA=angleA )

···

--
Daniel Hyams
dhyams@...287...

Thanks for reporting this.
This is now fixed in the v1.0.x-maint branch and the master branch.

Regards,

-JJ

···

On Sat, Sep 17, 2011 at 10:33 AM, Daniel Hyams <dhyams@...287...> wrote:

In http://matplotlib.sourceforge.net/users/annotations_guide.html ,
about 1/3 of the way down, there is a little demonstrator for the
different arrowstyles ->, <-, ]-, etc.

Looking at the figure closely, there is no difference between the "-["
and "]-" styles.

The fix for this is in patches.py, around line 3276 (matplotlib
1.0.0). Comments in that routine clipped out for brevity.

class BracketA(_Bracket):
def __init__(self, widthA=1., lengthA=0.2, angleA=None):

       super\(ArrowStyle\.BracketA, self\)\.\_\_init\_\_\(None,True,
                widthA=widthA, lengthA=lengthA, angleA=angleA \)

If you flip the first two args of the __init__ call, that fixes it:

class BracketA(_Bracket):
def __init__(self, widthA=1., lengthA=0.2, angleA=None):

       super\(ArrowStyle\.BracketA, self\)\.\_\_init\_\_\(True,None,

############## flip these arguments
widthA=widthA, lengthA=lengthA, angleA=angleA )

--
Daniel Hyams
dhyams@...287...

------------------------------------------------------------------------------
BlackBerry&reg; DevCon Americas, Oct. 18-20, San Francisco, CA
http://p.sf.net/sfu/rim-devcon-copy2
_______________________________________________
Matplotlib-users mailing list
Matplotlib-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-users