Bug#994575: dipy: autopkgtest idea

2022-03-23 Thread Nilesh Patra
On Fri, 24 Dec 2021 19:15:51 +0530 Nilesh Patra  wrote:
> On checking, it stems from the fact that fury (python module) that dipy needs 
> is not in (test-)depends, and neither is it in the
> archive.
> Probably other tests also are missing some or the other module. I also see a 
> couple of circular imports that we need to fix.
> 
> This is my understanding ^^ so maybe packaging fury will help.

Hi, examples consume a awful lot of resources since they download * heaps * of 
data and process it.
I do not think it would be very suitable for our runners;

hence for now I am disabling this test. However the pytest during autopkgtest 
test works and should do
the job for "autopkgtest for dipy", IMO. I will be closing this bug in the next 
upload.

Regards,
Nilesh


signature.asc
Description: PGP signature


Bug#994575: dipy: autopkgtest idea

2021-12-24 Thread Nilesh Patra
On Fri, 17 Dec 2021 16:34:02 +0100 Andreas Tille  wrote:
> Am Thu, Dec 16, 2021 at 09:10:45PM +0100 schrieb Andreas Tille:
> > Hi Étienne,
> > 
> > Am Fri, Sep 17, 2021 at 11:11:03PM +0200 schrieb Étienne Mollier:
> > > While working on #983840, I needed to do some testing of the
> > > package, and notice examples in doc/examples/.  Maybe they could
> > > be a source of inspiration for building an autopkgtest test
> > > suite?
> > 
> > I think the idea itself is good but the realisation has some flaws.
> > I've commited some code implementing the idea to Git but it seems we
> > need to exclude lots of those examples as you can see for instance here
> > in the Salsa CI
> > 
> >https://salsa.debian.org/med-team/dipy/-/jobs/2285615
> > 
> > May be running the build-time nosetest is the easier approach to an
> > autopkgtest.
> 
> Thanks for porting to pytest.  I've added the pytest test to autopkgtest
> but all tests are failing:
> 
> https://salsa.debian.org/med-team/dipy/-/pipelines/328590
> 
> Any idea what might be wrong here?

I ran it locally, for me it chokes with:

| autopkgtest [18:58:22]: test command1: xvfb-run --auto-servernum 
--server-num=20 -s "-screen 0 1024x768x24 -ac +extension GLX +render -noreset" 
sh debian/tests/run-unit-test
| autopkgtest [18:58:22]: test command1: [---
| Skip quick_start.py
| Run brain_extraction_dwi.py
| 100%|██| 589/589 [00:52<00:00, 11.30 MB/s]
| Skip reconst_csa_parallel.py
| Skip reconst_csa.py
| Skip reconst_csd_parallel.py
| Skip reconst_csd.py
| Run reconst_forecast.py
| /usr/lib/python3/dist-packages/dipy/viz/__init__.py:31: UserWarning: You do 
not have FURY installed. Some visualization functions might not work for you. 
For installation instructions, please visit: https://fury.gl/
|  warnings.warn(
| Traceback (most recent call last):
|   File 
"/tmp/autopkgtest.QurZey/autopkgtest_tmp/examples/reconst_forecast.py", line 
16, in 
| from dipy.viz import actor, window
| ImportError: cannot import name 'actor' from 'dipy.viz' 
(/usr/lib/python3/dist-packages/dipy/viz/__init__.py)

On checking, it stems from the fact that fury (python module) that dipy needs 
is not in (test-)depends, and neither is it in the
archive.
Probably other tests also are missing some or the other module. I also see a 
couple of circular imports that we need to fix.

This is my understanding ^^ so maybe packaging fury will help.

Regards,
Nilesh


signature.asc
Description: PGP signature


Bug#994575: dipy: autopkgtest idea

2021-12-17 Thread Andreas Tille
Am Thu, Dec 16, 2021 at 09:10:45PM +0100 schrieb Andreas Tille:
> Hi Étienne,
> 
> Am Fri, Sep 17, 2021 at 11:11:03PM +0200 schrieb Étienne Mollier:
> > While working on #983840, I needed to do some testing of the
> > package, and notice examples in doc/examples/.  Maybe they could
> > be a source of inspiration for building an autopkgtest test
> > suite?
> 
> I think the idea itself is good but the realisation has some flaws.
> I've commited some code implementing the idea to Git but it seems we
> need to exclude lots of those examples as you can see for instance here
> in the Salsa CI
> 
>https://salsa.debian.org/med-team/dipy/-/jobs/2285615
> 
> May be running the build-time nosetest is the easier approach to an
> autopkgtest.

Thanks for porting to pytest.  I've added the pytest test to autopkgtest
but all tests are failing:

https://salsa.debian.org/med-team/dipy/-/pipelines/328590

Any idea what might be wrong here?

Kind regards

  Andreas.
 

-- 
http://fam-tille.de



Bug#994575: dipy: autopkgtest idea

2021-12-16 Thread Andreas Tille
Hi Étienne,

Am Fri, Sep 17, 2021 at 11:11:03PM +0200 schrieb Étienne Mollier:
> While working on #983840, I needed to do some testing of the
> package, and notice examples in doc/examples/.  Maybe they could
> be a source of inspiration for building an autopkgtest test
> suite?

I think the idea itself is good but the realisation has some flaws.
I've commited some code implementing the idea to Git but it seems we
need to exclude lots of those examples as you can see for instance here
in the Salsa CI

   https://salsa.debian.org/med-team/dipy/-/jobs/2285615

May be running the build-time nosetest is the easier approach to an
autopkgtest.

Kind regards

  Andreas.

-- 
http://fam-tille.de



Bug#994575: dipy: autopkgtest idea

2021-09-17 Thread Étienne Mollier
Source: dipy
Version: 1.3.0-3
Severity: wishlist
Tags: newcomer

Dear Maintainer,

While working on #983840, I needed to do some testing of the
package, and notice examples in doc/examples/.  Maybe they could
be a source of inspiration for building an autopkgtest test
suite?

Have a nice day,  :)
-- 
Étienne Mollier 
Fingerprint:  8f91 b227 c7d6 f2b1 948c  8236 793c f67e 8f0d 11da
Sent from /dev/pts/7, please excuse my verbosity.


signature.asc
Description: PGP signature