[matplotlib-devel] Running tests on latest master

2012-08-28 Thread Nelle Varoquaux
Hello,

I've just checked out the source code from github, and I ran the
tests. Unfortunately, I get loads of errors (more than 1k), most of
them such as the following:


==
ERROR: matplotlib.tests.test_text.test_multiline.test
--
Traceback (most recent call last):
  File 
"/bioinfo/users/nvaroqua/Projects/matplotlib/env/lib/python2.6/site-packages/nose/case.py",
line 197, in runTest
self.test(*self.arg)
  File 
"/bioinfo/users/nvaroqua/Projects/matplotlib/env/lib/python2.6/site-packages/matplotlib/testing/decorators.py",
line 39, in failer
result = f(*args, **kwargs)
  File 
"/bioinfo/users/nvaroqua/Projects/matplotlib/env/lib/python2.6/site-packages/matplotlib/testing/decorators.py",
line 148, in do_test
self._tol, in_decorator=True)
  File 
"/bioinfo/users/nvaroqua/Projects/matplotlib/env/lib/python2.6/site-packages/matplotlib/testing/compare.py",
line 311, in compare_images
h1h = np.histogram(h1p, bins=bins)[0]
NameError: global name 'bins' is not defined


I've tried to have a look at the code, and indeed 'bins' is not
defined in this method. I'm not sure if this could be an installation
problem. Any idea how I could fix that ?

Many thanks,
N

PS: the developpers mailing list is *really* hard to find.

--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


Re: [matplotlib-devel] Running tests on latest master

2012-08-28 Thread Benjamin Root
On Tue, Aug 28, 2012 at 9:47 AM, Nelle Varoquaux
wrote:

> Hello,
>
> I've just checked out the source code from github, and I ran the
> tests. Unfortunately, I get loads of errors (more than 1k), most of
> them such as the following:
>
>
Looks like you triggered a code-path that most of us haven't encountered.
The code in question checks for which version of numpy is installed and
uses the faster bincount if it is available and falls back to the older
testing code if it isn't available.  Unfortunately, the older code seems to
have been bastardized in the process.  This should defintely be fixed.

I might have a moment today to submit a PR for this.

Ben Root
--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


Re: [matplotlib-devel] Running tests on latest master

2012-08-28 Thread Benjamin Root
On Tue, Aug 28, 2012 at 9:58 AM, Benjamin Root  wrote:

>
>
> On Tue, Aug 28, 2012 at 9:47 AM, Nelle Varoquaux <
> [email protected]> wrote:
>
>> Hello,
>>
>> I've just checked out the source code from github, and I ran the
>> tests. Unfortunately, I get loads of errors (more than 1k), most of
>> them such as the following:
>>
>>
> Looks like you triggered a code-path that most of us haven't encountered.
> The code in question checks for which version of numpy is installed and
> uses the faster bincount if it is available and falls back to the older
> testing code if it isn't available.  Unfortunately, the older code seems to
> have been bastardized in the process.  This should defintely be fixed.
>
> I might have a moment today to submit a PR for this.
>
> Ben Root
>
>

Heh, looks like it was a typo in line 305: change "ns" to "bins"

Ben
--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


Re: [matplotlib-devel] Running tests on latest master

2012-08-28 Thread Nelle Varoquaux
On 28 August 2012 16:01, Benjamin Root  wrote:
>
>
> On Tue, Aug 28, 2012 at 9:58 AM, Benjamin Root  wrote:
>>
>>
>>
>> On Tue, Aug 28, 2012 at 9:47 AM, Nelle Varoquaux
>>  wrote:
>>>
>>> Hello,
>>>
>>> I've just checked out the source code from github, and I ran the
>>> tests. Unfortunately, I get loads of errors (more than 1k), most of
>>> them such as the following:
>>>
>>
>> Looks like you triggered a code-path that most of us haven't encountered.
>> The code in question checks for which version of numpy is installed and uses
>> the faster bincount if it is available and falls back to the older testing
>> code if it isn't available.  Unfortunately, the older code seems to have
>> been bastardized in the process.  This should defintely be fixed.
>>
>> I might have a moment today to submit a PR for this.
>>
>> Ben Root
>>
>
>
> Heh, looks like it was a typo in line 305: change "ns" to "bins"

I have flake8 running automatically in my text editors, and I have a
bunch of warning. On the testing/compare.py files I've got:

compare.py|11| W402 'image_util' imported but unused
compare.py|16| W402 'math' imported but unused
compare.py|17| W402 'operator' imported but unused
compare.py|23| W402 'reduce' imported but unused
compare.py|305| W806 local variable 'ns' is assigned to but never used
compare.py|311| W802 undefined name 'bins'
compare.py|312| W802 undefined name 'bins'

and a bunch of pep8 warning. Are you guys interested in patches fixing
those ? It would then be easier to spot errors such as the previous
one.

Thanks,
N

>
> Ben

--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


Re: [matplotlib-devel] Running tests on latest master

2012-08-28 Thread Benjamin Root
On Tue, Aug 28, 2012 at 10:06 AM, Nelle Varoquaux  wrote:

> On 28 August 2012 16:01, Benjamin Root  wrote:
> >
> >
> > On Tue, Aug 28, 2012 at 9:58 AM, Benjamin Root  wrote:
> >>
> >>
> >>
> >> On Tue, Aug 28, 2012 at 9:47 AM, Nelle Varoquaux
> >>  wrote:
> >>>
> >>> Hello,
> >>>
> >>> I've just checked out the source code from github, and I ran the
> >>> tests. Unfortunately, I get loads of errors (more than 1k), most of
> >>> them such as the following:
> >>>
> >>
> >> Looks like you triggered a code-path that most of us haven't
> encountered.
> >> The code in question checks for which version of numpy is installed and
> uses
> >> the faster bincount if it is available and falls back to the older
> testing
> >> code if it isn't available.  Unfortunately, the older code seems to have
> >> been bastardized in the process.  This should defintely be fixed.
> >>
> >> I might have a moment today to submit a PR for this.
> >>
> >> Ben Root
> >>
> >
> >
> > Heh, looks like it was a typo in line 305: change "ns" to "bins"
>
> I have flake8 running automatically in my text editors, and I have a
> bunch of warning. On the testing/compare.py files I've got:
>
> compare.py|11| W402 'image_util' imported but unused
> compare.py|16| W402 'math' imported but unused
> compare.py|17| W402 'operator' imported but unused
> compare.py|23| W402 'reduce' imported but unused
> compare.py|305| W806 local variable 'ns' is assigned to but never used
> compare.py|311| W802 undefined name 'bins'
> compare.py|312| W802 undefined name 'bins'
>
> and a bunch of pep8 warning. Are you guys interested in patches fixing
> those ? It would then be easier to spot errors such as the previous
> one.
>
> Thanks,
> N
>
>
Wouldn't hurt.  We usually do PEP8 cleanups right before RC releases
anyway.  The testing code tends to get ignored, so any sort of attention
here would be great.

Ben Root
--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel


[matplotlib-devel] documentation error at http://matplotlib.sourceforge.net/sampledoc/custom_look.html

2012-08-28 Thread keith.briggs
"TS Elliots' maxim" should be "T. S. Eliot's maxim" :)

Keith


--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/___
Matplotlib-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel