Re: [matplotlib-devel] Tab Characters in Python Code

2010-04-28 Thread Eric Firing
Mark Roddy wrote: > On Wed, Apr 28, 2010 at 12:30 PM, Mark Roddy wrote: >> On Wed, Apr 28, 2010 at 11:49 AM, Michael Droettboom wrote: >>> Thanks. Yes, I would consider tabs a bug. A patch is very welcome. >>> [...] > > I added a patch to the tracker: > https://sourceforge.net/tracker/?func=de

Re: [matplotlib-devel] Tab Characters in Python Code

2010-04-28 Thread Mark Roddy
On Wed, Apr 28, 2010 at 12:30 PM, Mark Roddy wrote: > On Wed, Apr 28, 2010 at 11:49 AM, Michael Droettboom wrote: >> Thanks.  Yes, I would consider tabs a bug.  A patch is very welcome. >> >> Mike >> >> Mark Roddy wrote: >>> >>> I'm trying to resolve an issue with my source base that uses >>> mat

Re: [matplotlib-devel] Tab Characters in Python Code

2010-04-28 Thread Mark Roddy
On Wed, Apr 28, 2010 at 11:49 AM, Michael Droettboom wrote: > Thanks.  Yes, I would consider tabs a bug.  A patch is very welcome. > > Mike > > Mark Roddy wrote: >> >> I'm trying to resolve an issue with my source base that uses >> matplotlib.  We run our CI tests with the '-tt' option which cause

Re: [matplotlib-devel] Tab Characters in Python Code

2010-04-28 Thread Michael Droettboom
Thanks. Yes, I would consider tabs a bug. A patch is very welcome. Mike Mark Roddy wrote: > I'm trying to resolve an issue with my source base that uses > matplotlib. We run our CI tests with the '-tt' option which causes > errors on mixed tabs/spaces, and this has caused our builds to start >

[matplotlib-devel] Tab Characters in Python Code

2010-04-28 Thread Mark Roddy
I'm trying to resolve an issue with my source base that uses matplotlib. We run our CI tests with the '-tt' option which causes errors on mixed tabs/spaces, and this has caused our builds to start failing since the inclusion of matploblib on one of our projects due to mixed use of tab and space ch

Re: [matplotlib-devel] Incomplete rendering of line plot

2010-04-28 Thread Michael Droettboom
I think this is due to improper use of the path.simplify_threshold value in the simplification code. It should have been squared since it's used in a 2-dimensional euclidean distance calculation. I have made the change to SVN r8280 and updated a few of the unit tests that are now more accurate