[Numpy-discussion] unit tests / developing numpy (Pauli Virtanen)

[Graeme B. Bell]

> 
> No, runtests builds the code and sets PYTHONPATH accordingly.
> 
> -- 
> Pauli Virtanen


Hello Pauli,

Thanks again for writing back.

I agree that may be what runtests.py is intended to do, but it is unfortunately not what it actually does in its default configuration, at least on my computer. I got burned by this a few nights ago. 

$ pwd
/home/X/github/numpy

$ more numpy/version.py
# THIS FILE IS GENERATED FROM NUMPY SETUP.PY
short_version = '1.8.0'

$ echo $PYTHONPATH

$ python runtests.py 
Building, see build.log...
Build OK
Running unit tests for numpy
NumPy version 1.7.1
NumPy is installed in /usr/lib64/python2.7/site-packages/numpy
Python version 2.7.3 (default, Aug  9 2012, 17:23:57) [GCC 4.7.1 20120720 (Red Hat 4.7.1-5)]
nose version 1.3.0

*note the version number and directory used by runtests.py*

It reported 100% tests passed (unsurprising since it was testing the release version!). In reality, the current directory at that time failed a test when I pushed it to the main repository.

Can you suggest anything that I may be doing wrong here?

Graeme


On Jul 25, 2013, at 7:00 PM, numpy-discussion-request at scipy.org wrote:

> Send NumPy-Discussion mailing list submissions to
> 	numpy-discussion at scipy.org
> 
> To subscribe or unsubscribe via the World Wide Web, visit
> 	http://mail.scipy.org/mailman/listinfo/numpy-discussion
> or, via email, send a message with subject or body 'help' to
> 	numpy-discussion-request at scipy.org
> 
> You can reach the person managing the list at
> 	numpy-discussion-owner at scipy.org
> 
> When replying, please edit your Subject line so it is more specific
> than "Re: Contents of NumPy-Discussion digest..."
> 
> 
> Today's Topics:
> 
>   1. Re: add .H attribute? (Chris Barker - NOAA Federal)
>   2. Re: unit tests / developing numpy (Graeme B. Bell)
>   3. Re: unit tests / developing numpy (Pauli Virtanen)
> 
> 
> ----------------------------------------------------------------------
> 
> Message: 1
> Date: Thu, 25 Jul 2013 07:49:44 -0700
> From: Chris Barker - NOAA Federal <chris.barker at noaa.gov>
> Subject: Re: [Numpy-discussion] add .H attribute?
> To: Discussion of Numerical Python <numpy-discussion at scipy.org>
> Message-ID: <-29565752093052626 at unknownmsgid>
> Content-Type: text/plain; charset=UTF-8
> 
> On Jul 23, 2013, at 4:57 PM, Alan G Isaac <alan.isaac at gmail.com> wrote:
> 
>> Finally, I think (?) everyone (proponents and opponents)
>> would be happy if .H could provide access to an iterative
>> view of the conjugate transpose.
> 
> Except those of us that don't think numpy needs it at all.
> 
> But I'll call it a -0
> 
> -Chris
> 
> 
> ------------------------------
> 
> Message: 2
> Date: Thu, 25 Jul 2013 14:52:07 +0000
> From: "Graeme B. Bell" <grb at skogoglandskap.no>
> Subject: Re: [Numpy-discussion] unit tests / developing numpy
> To: "<numpy-discussion at scipy.org>" <numpy-discussion at scipy.org>
> Message-ID: <20D22199-8C71-49A0-8292-CEBBBB70B902 at skogoglandskap.no>
> Content-Type: text/plain; charset="us-ascii"
> 
> Nathaniel, Pauli:
> 
> Thanks for the suggestions!
> 
> = runtests.py is a nice solution, but unless you also set up your PYTHONPATH and install the code you've been working on, you're going to run with whichever version of numpy you have installed normally rather than the code you've just been working on (e.g. 1.7.1 rather than 1.8dev). 
> 
> Unfortunately, this caused me to submit a buggy set of commits, thinking they had passed the tests. 
> 
> 
> = env approach: thanks, I'll give that a try and compare it with the /tmp install approach.
> 
> 
> Can we add this to the dev workflow web pages?
> 
> Graeme.
> 
> ------------------------------
> 
> Message: 3
> Date: Thu, 25 Jul 2013 18:38:11 +0300
> From: Pauli Virtanen <pav at iki.fi>
> Subject: Re: [Numpy-discussion] unit tests / developing numpy
> To: numpy-discussion at scipy.org
> Message-ID: <ksrgos$60t$1 at ger.gmane.org>
> Content-Type: text/plain; charset=ISO-8859-1; format=flowed
> 
> 25.07.2013 17:52, Graeme B. Bell kirjoitti:
> [clip]
>> = runtests.py is a nice solution, but unless you also set up your
>> PYTHONPATH and install the code you've been working on, you're going
>> to run with whichever version of numpy you have installed normally
>> rather than the code you've just been working on (e.g. 1.7.1 rather 
> than 1.8dev).
> 
> No, runtests builds the code and sets PYTHONPATH accordingly.
> 
> -- 
> Pauli Virtanen
> 
> 
> 
> ------------------------------
> 
> _______________________________________________
> NumPy-Discussion mailing list
> NumPy-Discussion at scipy.org
> http://mail.scipy.org/mailman/listinfo/numpy-discussion
> 
> 
> End of NumPy-Discussion Digest, Vol 82, Issue 51
> ************************************************