Re: [matplotlib-devel] nxutils

2011-11-18 Thread Michael Droettboom 
nxutils has been removed from master because it is completely redundant 
to the Path functionality that has been in matplotlib since 0.98.  In 
the process of porting to Python 3, I felt it was important to reduce 
code duplication, because every additional line requires additional testing.


That said, there seems to be a lot of push back on this.  We can 
reinstate it, but I would suggest raising DeprecationWarnings for one 
release and then removing it entirely in the next.


Mike

On 11/18/2011 12:21 AM, Benjamin Root wrote:
Huh? Nxutils removed? Then how am I still using points_inside_poly? 
And, if I remember right, Path uses that to calculate contains().


Ben Root

On Thursday, November 17, 2011, Eric Firing > wrote:

> On 11/17/2011 10:19 AM, Michael Droettboom wrote:
>> Most of what was in nxutils has been superseded by things in Numpy, and
>> it makes more sense for it to be over there.
>>
>> In the case of points_inside_poly, you can use the Path object in
>> path.py and the "contains_point" method.
>>
>> Mike
>
> Mike,
>
> This, however, brings us back to the plea by Volker Blum:
>
> 
http://www.mail-archive.com/[email protected]/msg22669.html

>
> There is a real tension between the need to clean things up and simplify
> them, and users' desire for minimal loss of backwards compatibility.
> Personally, my instincts are in the "clean it up" camp, but a good
> balance has to be found.
>
> nxutils was definitely a vestige of an earlier era; but I don't think it
> went through any official, publicized, deprecation process, did it?
> Maybe it didn't need to; I don't know.  Perhaps we need to formulate and
> write down a deprecation policy.
>
> Eric
>
>>
>> On 11/17/2011 12:03 PM, James Evans wrote:
>>>
>>> All,
>>>
>>> I have not touched the code for several months, so it has taken me up
>>> until just now to realize that nxutils has been removed from the 
build.

>>>
>>> I there any real reason for this? Particularly when you consider that
>>> there are still functions present that use it and now they just fail.
>>>
>>> In particular I am referring to 'mlab.inside_poly'. In my case I was
>>> using 'nxutils.points_inside_poly' directly, but the end result is the
>>> same.
>>>
>>> Thanks,
>>>
>>> --James Evans
>>>
>>>
>>>
>>> 
--

>>> All the data continuously generated in your IT infrastructure
>>> contains a definitive record of customers, application performance,
>>> security threats, fraudulent activity, and more. Splunk takes this
>>> data and makes sense of it. IT sense. And common sense.
>>> http://p.sf.net/sfu/splunk-novd2d
>>>
>>>
>>> ___
>>> Matplotlib-devel mailing list
>>> [email protected] 


>>> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>>
>>
>>
>> 
--

>> All the data continuously generated in your IT infrastructure
>> contains a definitive record of customers, application performance,
>> security threats, fraudulent activity, and more. Splunk takes this
>> data and makes sense of it. IT sense. And common sense.
>> http://p.sf.net/sfu/splunk-novd2d
>>
>>
>>
>> ___
>> Matplotlib-devel mailing list
>> [email protected] 


>> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>
>
>
> 
--

> All the data continuously generated in your IT infrastructure
> contains a definitive record of customers, application performance,
> security threats, fraudulent activity, and more. Splunk takes this
> data and makes sense of it. IT sense. And common sense.
> http://p.sf.net/sfu/splunk-novd2d
> ___
> Matplotlib-devel mailing list
> [email protected] 


> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>


--
All the data continuously generated in your IT infrastructure
contains a definitive record of customers, application performance,
security threats, fraudulent activity, and more. Splunk takes this
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d


___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


--
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
secu

Re: [matplotlib-devel] nxutils

2011-11-18 Thread Michael Droettboom 
Perhaps another alternative is to just include a small compatibility 
module that would call the new functionality under the hood.


Mike

On 11/18/2011 09:07 AM, Michael Droettboom wrote:
nxutils has been removed from master because it is completely 
redundant to the Path functionality that has been in matplotlib since 
0.98.  In the process of porting to Python 3, I felt it was important 
to reduce code duplication, because every additional line requires 
additional testing.


That said, there seems to be a lot of push back on this.  We can 
reinstate it, but I would suggest raising DeprecationWarnings for one 
release and then removing it entirely in the next.


Mike

On 11/18/2011 12:21 AM, Benjamin Root wrote:
Huh? Nxutils removed? Then how am I still using points_inside_poly? 
And, if I remember right, Path uses that to calculate contains().


Ben Root

On Thursday, November 17, 2011, Eric Firing > wrote:

> On 11/17/2011 10:19 AM, Michael Droettboom wrote:
>> Most of what was in nxutils has been superseded by things in 
Numpy, and

>> it makes more sense for it to be over there.
>>
>> In the case of points_inside_poly, you can use the Path object in
>> path.py and the "contains_point" method.
>>
>> Mike
>
> Mike,
>
> This, however, brings us back to the plea by Volker Blum:
>
> 
http://www.mail-archive.com/[email protected]/msg22669.html

>
> There is a real tension between the need to clean things up and 
simplify

> them, and users' desire for minimal loss of backwards compatibility.
> Personally, my instincts are in the "clean it up" camp, but a good
> balance has to be found.
>
> nxutils was definitely a vestige of an earlier era; but I don't 
think it

> went through any official, publicized, deprecation process, did it?
> Maybe it didn't need to; I don't know.  Perhaps we need to 
formulate and

> write down a deprecation policy.
>
> Eric
>
>>
>> On 11/17/2011 12:03 PM, James Evans wrote:
>>>
>>> All,
>>>
>>> I have not touched the code for several months, so it has taken me up
>>> until just now to realize that nxutils has been removed from the 
build.

>>>
>>> I there any real reason for this? Particularly when you consider that
>>> there are still functions present that use it and now they just fail.
>>>
>>> In particular I am referring to 'mlab.inside_poly'. In my case I was
>>> using 'nxutils.points_inside_poly' directly, but the end result 
is the

>>> same.
>>>
>>> Thanks,
>>>
>>> --James Evans
>>>
>>>
>>>
>>> 
--

>>> All the data continuously generated in your IT infrastructure
>>> contains a definitive record of customers, application performance,
>>> security threats, fraudulent activity, and more. Splunk takes this
>>> data and makes sense of it. IT sense. And common sense.
>>> http://p.sf.net/sfu/splunk-novd2d
>>>
>>>
>>> ___
>>> Matplotlib-devel mailing list
>>> [email protected] 


>>> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>>
>>
>>
>> 
--

>> All the data continuously generated in your IT infrastructure
>> contains a definitive record of customers, application performance,
>> security threats, fraudulent activity, and more. Splunk takes this
>> data and makes sense of it. IT sense. And common sense.
>> http://p.sf.net/sfu/splunk-novd2d
>>
>>
>>
>> ___
>> Matplotlib-devel mailing list
>> [email protected] 


>> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>
>
>
> 
--

> All the data continuously generated in your IT infrastructure
> contains a definitive record of customers, application performance,
> security threats, fraudulent activity, and more. Splunk takes this
> data and makes sense of it. IT sense. And common sense.
> http://p.sf.net/sfu/splunk-novd2d
> ___
> Matplotlib-devel mailing list
> [email protected] 


> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>


--
All the data continuously generated in your IT infrastructure
contains a definitive record of customers, application performance,
security threats, fraudulent activity, and more. Splunk takes this
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d


___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel




-

Re: [matplotlib-devel] nxutils

2011-11-18 Thread James Evans 
I was just shocked to find the source code still present, just not compiled
during the build step and at least one completely broken function call still
referencing the un-built module and no apparent reason for removal. 

 

I have updated mlab.inside_poly to use Path instead and will submit it later
today.

 

--James

 

 

From: Michael Droettboom [mailto:[email protected]] 
Sent: Friday, November 18, 2011 6:23 AM
To: [email protected]
Subject: Re: [matplotlib-devel] nxutils

 

Perhaps another alternative is to just include a small compatibility module
that would call the new functionality under the hood.

Mike

On 11/18/2011 09:07 AM, Michael Droettboom wrote: 

nxutils has been removed from master because it is completely redundant to
the Path functionality that has been in matplotlib since 0.98.  In the
process of porting to Python 3, I felt it was important to reduce code
duplication, because every additional line requires additional testing.

That said, there seems to be a lot of push back on this.  We can reinstate
it, but I would suggest raising DeprecationWarnings for one release and then
removing it entirely in the next.

Mike

On 11/18/2011 12:21 AM, Benjamin Root wrote: 

Huh? Nxutils removed? Then how am I still using points_inside_poly? And, if
I remember right, Path uses that to calculate contains().

Ben Root

On Thursday, November 17, 2011, Eric Firing  wrote:
> On 11/17/2011 10:19 AM, Michael Droettboom wrote:
>> Most of what was in nxutils has been superseded by things in Numpy, and
>> it makes more sense for it to be over there.
>>
>> In the case of points_inside_poly, you can use the Path object in
>> path.py and the "contains_point" method.
>>
>> Mike
>
> Mike,
>
> This, however, brings us back to the plea by Volker Blum:
>
>
http://www.mail-archive.com/[email protected]/msg22669.
html
>
> There is a real tension between the need to clean things up and simplify
> them, and users' desire for minimal loss of backwards compatibility.
> Personally, my instincts are in the "clean it up" camp, but a good
> balance has to be found.
>
> nxutils was definitely a vestige of an earlier era; but I don't think it
> went through any official, publicized, deprecation process, did it?
> Maybe it didn't need to; I don't know.  Perhaps we need to formulate and
> write down a deprecation policy.
>
> Eric
>
>>
>> On 11/17/2011 12:03 PM, James Evans wrote:
>>>
>>> All,
>>>
>>> I have not touched the code for several months, so it has taken me up
>>> until just now to realize that nxutils has been removed from the build.
>>>
>>> I there any real reason for this? Particularly when you consider that
>>> there are still functions present that use it and now they just fail.
>>>
>>> In particular I am referring to 'mlab.inside_poly'. In my case I was
>>> using 'nxutils.points_inside_poly' directly, but the end result is the
>>> same.
>>>
>>> Thanks,
>>>
>>> --James Evans
>>>
>>>
>>>
>>>

--
>>> All the data continuously generated in your IT infrastructure
>>> contains a definitive record of customers, application performance,
>>> security threats, fraudulent activity, and more. Splunk takes this
>>> data and makes sense of it. IT sense. And common sense.
>>> http://p.sf.net/sfu/splunk-novd2d
>>>
>>>
>>> ___
>>> Matplotlib-devel mailing list
>>> [email protected]
>>> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>>
>>
>>
>>

--
>> All the data continuously generated in your IT infrastructure
>> contains a definitive record of customers, application performance,
>> security threats, fraudulent activity, and more. Splunk takes this
>> data and makes sense of it. IT sense. And common sense.
>> http://p.sf.net/sfu/splunk-novd2d
>>
>>
>>
>> ___
>> Matplotlib-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>
>
>
>

--
> All the data continuously generated in your IT infrastructure
> contains a definitive record of customers, application performance,
> security threats, fraudulent activity, and more. Splunk takes this
> data and makes sense of it. IT sense. And common sense.
> http://p.sf.net/sfu/splunk-novd2d
> ___
> Matplotlib-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
> 





--
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and mo

[matplotlib-devel] Subject: Re: Another colormap

2011-11-18 Thread Pim Schellart 
Dear Tony,

I tried the example with several colormaps (have you checked out
cubehelix which nicely resembles the grey scale visual intensity
distribution in color) and I definitely agree that it would be good
for matplotlib to switch to a more sensible default color map. My
personal vote goes to coolwarm which has well defined behaviour and is
suitable (e.g. looks nice) for a wide range of applications.

Kind regards,

Pim Schellart

P.S. Although cubehelix also has well defined behaviour it is less
optimal as a default since it does not look nice in all use cases (but
is very good in some, particularly for cases where the percieved
intensity distribution needs to be the same when viewed on screen and
printed in black and white).

--
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Re: [matplotlib-devel] nxutils

2011-11-18 Thread Michael Droettboom 
I'm sorry this fell through the cracks.  It was removed from the build 
because it does not currently build on Python 3.x, and then when I saw 
the functionality was redundant, I think it necessary to invest effort 
porting it.  None of the test suite uses this code.  The source should 
have been removed at the same time, as well as updating the mlab usage.  
You'll see I did add a "TODO" above it that should have been addressed 
before merging into master.


This shows a hole in the test coverage -- we should add a test that uses 
mlab.inside_poly as part of fixing this.


Mike

On 11/18/2011 10:48 AM, James Evans wrote:


I was just shocked to find the source code still present, just not 
compiled during the build step and at least one completely broken 
function call still referencing the un-built module and no apparent 
reason for removal.


I have updated mlab.inside_poly to use Path instead and will submit it 
later today.


--James

*From:*Michael Droettboom [mailto:[email protected]]
*Sent:* Friday, November 18, 2011 6:23 AM
*To:* [email protected]
*Subject:* Re: [matplotlib-devel] nxutils

Perhaps another alternative is to just include a small compatibility 
module that would call the new functionality under the hood.


Mike

On 11/18/2011 09:07 AM, Michael Droettboom wrote:

nxutils has been removed from master because it is completely 
redundant to the Path functionality that has been in matplotlib since 
0.98.  In the process of porting to Python 3, I felt it was important 
to reduce code duplication, because every additional line requires 
additional testing.


That said, there seems to be a lot of push back on this.  We can 
reinstate it, but I would suggest raising DeprecationWarnings for one 
release and then removing it entirely in the next.


Mike

On 11/18/2011 12:21 AM, Benjamin Root wrote:

Huh? Nxutils removed? Then how am I still using points_inside_poly? 
And, if I remember right, Path uses that to calculate contains().


Ben Root

On Thursday, November 17, 2011, Eric Firing > wrote:

> On 11/17/2011 10:19 AM, Michael Droettboom wrote:
>> Most of what was in nxutils has been superseded by things in Numpy, and
>> it makes more sense for it to be over there.
>>
>> In the case of points_inside_poly, you can use the Path object in
>> path.py and the "contains_point" method.
>>
>> Mike
>
> Mike,
>
> This, however, brings us back to the plea by Volker Blum:
>
> 
http://www.mail-archive.com/[email protected]/msg22669.html

>
> There is a real tension between the need to clean things up and simplify
> them, and users' desire for minimal loss of backwards compatibility.
> Personally, my instincts are in the "clean it up" camp, but a good
> balance has to be found.
>
> nxutils was definitely a vestige of an earlier era; but I don't think it
> went through any official, publicized, deprecation process, did it?
> Maybe it didn't need to; I don't know.  Perhaps we need to formulate and
> write down a deprecation policy.
>
> Eric
>
>>
>> On 11/17/2011 12:03 PM, James Evans wrote:
>>>
>>> All,
>>>
>>> I have not touched the code for several months, so it has taken me up
>>> until just now to realize that nxutils has been removed from the 
build.

>>>
>>> I there any real reason for this? Particularly when you consider that
>>> there are still functions present that use it and now they just fail.
>>>
>>> In particular I am referring to 'mlab.inside_poly'. In my case I was
>>> using 'nxutils.points_inside_poly' directly, but the end result is the
>>> same.
>>>
>>> Thanks,
>>>
>>> --James Evans
>>>
>>>
>>>
>>> 
--

>>> All the data continuously generated in your IT infrastructure
>>> contains a definitive record of customers, application performance,
>>> security threats, fraudulent activity, and more. Splunk takes this
>>> data and makes sense of it. IT sense. And common sense.
>>> http://p.sf.net/sfu/splunk-novd2d
>>>
>>>
>>> ___
>>> Matplotlib-devel mailing list
>>> [email protected] 


>>> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>>
>>
>>
>> 
--

>> All the data continuously generated in your IT infrastructure
>> contains a definitive record of customers, application performance,
>> security threats, fraudulent activity, and more. Splunk takes this
>> data and makes sense of it. IT sense. And common sense.
>> http://p.sf.net/sfu/splunk-novd2d
>>
>>
>>
>> ___
>> Matplotlib-devel mailing list
>> [email protected] 


>> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
>
>
>
> 
---