Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-25 Thread Ryan May
Eric Firing wrote: John Hunter wrote: On Thu, Jul 24, 2008 at 10:00 PM, Ryan May [EMAIL PROTECTED] wrote: What else is confusing is how that relates to DPI. When I change the figure's dpi, using set_dpi, (and redraw), I get physically *bigger* barbs. To me, if I'm actually specifying

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-25 Thread John Hunter
On Fri, Jul 25, 2008 at 5:47 PM, Ryan May [EMAIL PROTECTED] wrote: Ok, I think I found our problem, at line 859 of backend_ps.py (inside _print_ps()): self.figure.set_dpi(72) # Override the dpi kwarg dpi = kwargs.get(dpi, 72) The problem here is that while it sets the figure

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-25 Thread Eric Firing
Ryan May wrote: John Hunter wrote: On Thu, Jul 24, 2008 at 10:00 PM, Ryan May [EMAIL PROTECTED] wrote: What else is confusing is how that relates to DPI. When I change the figure's dpi, using set_dpi, (and redraw), I get physically *bigger* barbs. To me, if I'm actually specifying

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-25 Thread Eric Firing
John Hunter wrote: On Fri, Jul 25, 2008 at 5:47 PM, Ryan May [EMAIL PROTECTED] wrote: Ok, I think I found our problem, at line 859 of backend_ps.py (inside _print_ps()): self.figure.set_dpi(72) # Override the dpi kwarg dpi = kwargs.get(dpi, 72) The problem here is that

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-25 Thread John Hunter
On Fri, Jul 25, 2008 at 7:11 PM, Ryan May [EMAIL PROTECTED] wrote: try: from hashlib import md5 except ImportError: from md5 import md5 Looks good to me -- thanks for the offer to fix this. In general, we only actively want to be supporting 2 python versions at a time, but if there is

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-25 Thread Ryan May
John Hunter wrote: On Fri, Jul 25, 2008 at 6:42 PM, Eric Firing [EMAIL PROTECTED] wrote: This is the kind of thing I would want to look at and test *very* carefully, or know that someone like John or Mike has done so--which perhaps one of them can do quickly. I have looked at this dpi

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-25 Thread Eric Firing
John Hunter wrote: On Fri, Jul 25, 2008 at 6:42 PM, Eric Firing [EMAIL PROTECTED] wrote: This is the kind of thing I would want to look at and test *very* carefully, or know that someone like John or Mike has done so--which perhaps one of them can do quickly. I have looked at this dpi

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-24 Thread John Hunter
On Wed, Jul 23, 2008 at 10:05 PM, Ryan May [EMAIL PROTECTED] wrote: grumble Ok, it fixes the problem if I pass dpi=72 to savefig(). Curiously, passing dpi=72 to Figure() does not have the same effect. So now how do I That is because savefig has its own dpi, which overrides the figure dpi.

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-24 Thread Eric Firing
John Hunter wrote: On Wed, Jul 23, 2008 at 10:05 PM, Ryan May [EMAIL PROTECTED] wrote: grumble Ok, it fixes the problem if I pass dpi=72 to savefig(). Curiously, passing dpi=72 to Figure() does not have the same effect. So now how do I That is because savefig has its own dpi, which

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-24 Thread Ryan May
Eric Firing wrote: John Hunter wrote: On Wed, Jul 23, 2008 at 10:05 PM, Ryan May [EMAIL PROTECTED] wrote: grumble Ok, it fixes the problem if I pass dpi=72 to savefig(). Curiously, passing dpi=72 to Figure() does not have the same effect. So now how do I That is because savefig has its

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-24 Thread John Hunter
On Thu, Jul 24, 2008 at 10:00 PM, Ryan May [EMAIL PROTECTED] wrote: What else is confusing is how that relates to DPI. When I change the figure's dpi, using set_dpi, (and redraw), I get physically *bigger* barbs. To me, if I'm actually specifying pixels, there's no way that they should get

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-24 Thread Eric Firing
John Hunter wrote: On Thu, Jul 24, 2008 at 10:00 PM, Ryan May [EMAIL PROTECTED] wrote: What else is confusing is how that relates to DPI. When I change the figure's dpi, using set_dpi, (and redraw), I get physically *bigger* barbs. To me, if I'm actually specifying pixels, there's no way

Re: [matplotlib-devel] Wind Barbs

2008-07-23 Thread Eric Firing
Ryan May wrote: Ok, I've committed my wind barbs stuff in SVN. Anyone interested, go ahead an hammer on it. Should I post this over on matplotlib-users, or does experience show that the SVN crowd all read here? (This is what you get for having a n00b developer :) ) We routinely

Re: [matplotlib-devel] Wind Barbs

2008-07-23 Thread Eric Firing
Ryan May wrote: Ok, I've committed my wind barbs stuff in SVN. Anyone interested, go ahead an hammer on it. Very nice! Comments on the example, barb_demo.py: 1) In your third panel, you put the args after the kwargs. I had no idea this was even legal--it's the first time I have seen

Re: [matplotlib-devel] Wind Barbs

2008-07-23 Thread Ryan May
Eric Firing wrote: Ryan May wrote: Ok, I've committed my wind barbs stuff in SVN. Anyone interested, go ahead an hammer on it. Very nice! Comments on the example, barb_demo.py: 1) In your third panel, you put the args after the kwargs. I had no idea this was even legal--it's the

Re: [matplotlib-devel] Wind Barbs

2008-07-23 Thread Ryan May
Eric Firing wrote: Ryan May wrote: Ok, I've committed my wind barbs stuff in SVN. Anyone interested, go ahead an hammer on it. Very nice! Comments on the example, barb_demo.py: 1) In your third panel, you put the args after the kwargs. I had no idea this was even legal--it's the

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-23 Thread Ryan May
John Hunter wrote: On Sat, Jul 19, 2008 at 11:09 PM, Ryan May [EMAIL PROTECTED] wrote: The only issue I've seen is that scaling with PS is way too big. I've attached ps and pdf files from the same run to show the problem. The only thing I can think of is since you are using a identity

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-22 Thread Ryan May
How do you guys manage committing only parts of your working copy, especially when you want to commit part of a file? I figure there's got to be a better way than multiple SVN checkouts and manually editing diffs. svn should do this automagically; it only commits the diff from your current

[matplotlib-devel] Wind Barbs

2008-07-22 Thread Ryan May
Ok, I've committed my wind barbs stuff in SVN. Anyone interested, go ahead an hammer on it. Should I post this over on matplotlib-users, or does experience show that the SVN crowd all read here? (This is what you get for having a n00b developer :) ) Jeff, does a method need to be added to

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-21 Thread Ryan May
Michael Droettboom wrote: Ryan May wrote: 5) I added an empty circle marker for low wind speeds (vector magnitudes). Accomplishing having the unfilled circle while having the barbs filled involved a bit of a elegant hack. Using the set of vertices that draws the CirclePolygon, I add an

Re: [matplotlib-devel] Wind Barbs Full Patch

2008-07-20 Thread John Hunter
On Sat, Jul 19, 2008 at 11:09 PM, Ryan May [EMAIL PROTECTED] wrote: The only issue I've seen is that scaling with PS is way too big. I've attached ps and pdf files from the same run to show the problem. The only thing I can think of is since you are using a identity transform and drawing in

Re: [matplotlib-devel] (Wind) Barbs

2008-07-17 Thread Ryan May
John Hunter wrote: On Tue, Jul 15, 2008 at 5:37 PM, Ryan May [EMAIL PROTECTED] wrote: I welcome any comments/criticism to help improve this. Hey Ryan, I have looked at this code briefly and have a few minor comments. I think Eric, who did the bulk of the current quiver implementation,

Re: [matplotlib-devel] (Wind) Barbs

2008-07-17 Thread Ryan May
Jeff Whitaker wrote: Ryan May wrote: Hi, I've got (what seems to me) a nice clean, self-contained implementation of wind barbs plots. I'd like to see if I can get this into matplotlib, as it would be very useful to the meteorology community. I've borrowed heavily from Quiver for

Re: [matplotlib-devel] (Wind) Barbs

2008-07-16 Thread Michael Droettboom
John Hunter wrote: On Tue, Jul 15, 2008 at 5:37 PM, Ryan May [EMAIL PROTECTED] wrote: The 2nd alternative, which I haven't explored, is to set the edgecolor equal to the facecolor and support colormapping of the edgecolors. FWIW, pcolor and pcolormesh have also needed this

Re: [matplotlib-devel] (Wind) Barbs

2008-07-16 Thread Jeff Whitaker
Ryan May wrote: Hi, I've got (what seems to me) a nice clean, self-contained implementation of wind barbs plots. I'd like to see if I can get this into matplotlib, as it would be very useful to the meteorology community. I've borrowed heavily from Quiver for rounding out rough edges (like

[matplotlib-devel] (Wind) Barbs

2008-07-15 Thread Ryan May
Hi, I've got (what seems to me) a nice clean, self-contained implementation of wind barbs plots. I'd like to see if I can get this into matplotlib, as it would be very useful to the meteorology community. I've borrowed heavily from Quiver for rounding out rough edges (like multiple calling

Re: [matplotlib-devel] (Wind) Barbs

2008-07-15 Thread John Hunter
On Tue, Jul 15, 2008 at 5:37 PM, Ryan May [EMAIL PROTECTED] wrote: I welcome any comments/criticism to help improve this. Hey Ryan, I have looked at this code briefly and have a few minor comments. I think Eric, who did the bulk of the current quiver implementation, and as an oceanographer is