Actually, I have just realized that this code can be tested. It would be great ...
14 years, 8 months ago
(2009-08-13 19:59:41 UTC)
#4
Actually, I have just realized that this code can be tested. It would be great
if you could add a test, eg to enthought/mayavi/tests/test_array_source.py. We
are trying to have as much test coverage as possible. It has proven very useful.
Also, the existing tests are broken by your patch. I haven't had time to check
if it is just noise, or actual problems. You can run the test suite using
'mayavi2 -t'.
On 2009/08/13 19:59:41, Gael.Varoquaux wrote: > Also, the existing tests are broken by your patch. ...
14 years, 4 months ago
(2009-12-20 15:48:14 UTC)
#5
On 2009/08/13 19:59:41, Gael.Varoquaux wrote:
> Also, the existing tests are broken by your patch. I haven't had time to check
> if it is just noise, or actual problems. You can run the test suite using
> 'mayavi2 -t'.
OK, I have fixed these failures and checked the patch in.
Issue 104104: Add ImageChangeInformation filter to ArraySource to reliably set spacing and origin.
Created 14 years, 8 months ago by SJSwarts
Modified 9 years, 4 months ago
Reviewers: gael.varoquaux_normalesup.org, prabhu_aero.iitb.ac.in, Gael.Varoquaux
Base URL: https://svn.enthought.com/svn/enthought/Mayavi/trunk/
Comments: 0