[matplotlib-devel] barh vs. bar(orientation='horizontal')

2007-03-02 Thread Jouni K . Seppänen
Hi,

What is the reason for having both barh and bar, when the latter
accepts the orientation='horizontal' argument? I am asking because of
sf bug #1669506, which is about hist(orientation='horizontal') not
working because it passes a log kwarg to barh.

-- 
Jouni K. Seppänen
http://www.iki.fi/jks


-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


Re: [matplotlib-devel] barh vs. bar(orientation='horizontal')

2007-03-02 Thread Eric Firing
Jouni K. Seppänen wrote:
> Hi,
> 
> What is the reason for having both barh and bar, when the latter
> accepts the orientation='horizontal' argument? I am asking because of
> sf bug #1669506, which is about hist(orientation='horizontal') not
> working because it passes a log kwarg to barh.
> 

I didn't do it--but it looks like the reason is that having barh as a 
separate method permits a more natural order of arguments without 
introducing more complexity in the argument handling.

Vertical bars take left, height, width, bottom;
barh takes bottom, width, height, left.

Handling this difference with all possible combinations of *args and 
**kwargs would be complex; the present method, using a separate name 
(barh), is nice and simple.

Looks like barh just needs to take a **kwargs (which could replace most 
of the present listed kwargs; or add a log kwarg to the list) and pass 
it along to bar.  You are taking care of this?

Eric

-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


Re: [matplotlib-devel] barh vs. bar(orientation='horizontal')

2007-03-02 Thread John Hunter
On 3/2/07, Eric Firing <[EMAIL PROTECTED]> wrote:


> I didn't do it--but it looks like the reason is that having barh as a
> separate method permits a more natural order of arguments without
> introducing more complexity in the argument handling.

barh was originally added for compatibility with matlab and on a user
request.  I think the kwarg was added to bar to support easy calling
for functions that use bar (eg hist) but may also want to configure
the orientation.

-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


Re: [matplotlib-devel] barh vs. bar(orientation='horizontal')

2007-03-02 Thread Jouni K . Seppänen
Eric Firing <[EMAIL PROTECTED]> writes:

> Looks like barh just needs to take a **kwargs (which could replace most 
> of the present listed kwargs; or add a log kwarg to the list) and pass 
> it along to bar.  You are taking care of this?

I replaced most of the kwargs by a **kwargs dict in svn revision 3037.
This does change the behavior for people who were giving positional
arguments to barh, but they do get an error message.

-- 
Jouni K. Seppänen
http://www.iki.fi/jks


-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel