Bug#849177: Python-numpy transition breaks at least two packages

2016-12-23 Thread Ole Streicher
Hi all,

On 23.12.2016 11:19, Andreas Tille wrote:
> I'm also not sure and people who know better than me should feel free to
> close bug #849177.  I'd be really happy if there are better solutions to
> fix python-skimage in the next 36 hours.  My reading of the arguments in
> #849177 is that there are other reasons to revert the upgrade even
> without the additional problem - thus I was opening the bug report
> independently from other packages like python-skimage and python-skbio.

While the original FTBFS for skimage is solved now, I found another
problem with numpy 1.12 RC (and skimage) which I reported upstream:

https://github.com/numpy/numpy/issues/8413

Currently, this keeps up to get a version of skimage into Debian that
has a chance to migrate.

This is really very disappointing to run into those problems just a few
hours before the window for the stretch migration closes.

Sandro, could you please *immediately* rollback the numpy RC or at least
upload a version that otherwise solves all the regressions that are
still open? Please put a stable, working version of numpy that we all
can rely on. Time matters here.

Best regards

Ole (quite angry in the moment, sorry.)



Bug#849177: Python-numpy transition breaks at least two packages

2016-12-23 Thread Andreas Tille
Hi Ghislain,

On Fri, Dec 23, 2016 at 10:02:33AM +, Ghislain Vaillant wrote:
> On Fri, 2016-12-23 at 10:43 +0100, Andreas Tille wrote:
> > Hi,
> > 
> > I have cloned bug #848758 where I suggested to revert the python-numpy
> > transition which other posters agreed upon.  Besides breaking
> > python-skbio I spotted another package python-skimage which fails with:
> > 
> > 
> > ==
> > ERROR: skimage.filters.rank.tests.test_rank.test_all
> > --
> > Traceback (most recent call last):
> >   File "/usr/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
> > self.test(*self.arg)
> >   File 
> > "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/tests/test_rank.py",
> >  line 16, in test_all
> > check_all()
> >   File 
> > "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/_shared/testing.py",
> >  line 232, in inner
> > result = func(*args, **kwargs)
> >   File 
> > "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/tests/test_rank.py",
> >  line 89, in check_all
> > rank.windowed_histogram(image, selem))
> >   File 
> > "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
> >  line 986, in windowed_histogram
> > pixel_size=n_bins)
> >   File 
> > "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
> >  line 90, in _apply_vector_per_pixel
> > pixel_size=pixel_size)
> >   File 
> > "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
> >  line 53, in _handle_input
> > out = np.empty(image.shape+(pixel_size,), dtype=out_dtype)
> > TypeError: 'numpy.float64' object cannot be interpreted as an index
> > 
> > --
> > Ran 1383 tests in 181.697s
> 
> Sounds like the kind of bug upstream would be interested in getting
> fixed. This implementation must be relying on an implicit integer type
> for the overall shape of the filter. My initial guess would be that
> pixel_size ends-up being a float rather than an int here?

Yes, sure.  Any volunteer to submit the issue upstream?  Sorry, I will
not manage it in a sensible time frame.
 
> Not sure whether it is worth reverting numpy for this. From a quick
> search, it looks like skimage has had issues like this in the past.

I'm also not sure and people who know better than me should feel free to
close bug #849177.  I'd be really happy if there are better solutions to
fix python-skimage in the next 36 hours.  My reading of the arguments in
#849177 is that there are other reasons to revert the upgrade even
without the additional problem - thus I was opening the bug report
independently from other packages like python-skimage and python-skbio.

Kind regards

  Andreas.
-- 
http://fam-tille.de



Bug#849177: Python-numpy transition breaks at least two packages

2016-12-23 Thread Ghislain Vaillant
On Fri, 2016-12-23 at 10:43 +0100, Andreas Tille wrote:
> Hi,
> 
> I have cloned bug #848758 where I suggested to revert the python-numpy
> transition which other posters agreed upon.  Besides breaking
> python-skbio I spotted another package python-skimage which fails with:
> 
> 
> ==
> ERROR: skimage.filters.rank.tests.test_rank.test_all
> --
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
> self.test(*self.arg)
>   File 
> "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/tests/test_rank.py",
>  line 16, in test_all
> check_all()
>   File 
> "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/_shared/testing.py",
>  line 232, in inner
> result = func(*args, **kwargs)
>   File 
> "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/tests/test_rank.py",
>  line 89, in check_all
> rank.windowed_histogram(image, selem))
>   File 
> "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
>  line 986, in windowed_histogram
> pixel_size=n_bins)
>   File 
> "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
>  line 90, in _apply_vector_per_pixel
> pixel_size=pixel_size)
>   File 
> "/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
>  line 53, in _handle_input
> out = np.empty(image.shape+(pixel_size,), dtype=out_dtype)
> TypeError: 'numpy.float64' object cannot be interpreted as an index
> 
> --
> Ran 1383 tests in 181.697s

Sounds like the kind of bug upstream would be interested in getting
fixed. This implementation must be relying on an implicit integer type
for the overall shape of the filter. My initial guess would be that
pixel_size ends-up being a float rather than an int here?

Not sure whether it is worth reverting numpy for this. From a quick
search, it looks like skimage has had issues like this in the past.

Ghis



Bug#849177: Python-numpy transition breaks at least two packages

2016-12-23 Thread Andreas Tille
Hi,

I have cloned bug #848758 where I suggested to revert the python-numpy
transition which other posters agreed upon.  Besides breaking
python-skbio I spotted another package python-skimage which fails with:


==
ERROR: skimage.filters.rank.tests.test_rank.test_all
--
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
self.test(*self.arg)
  File 
"/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/tests/test_rank.py",
 line 16, in test_all
check_all()
  File 
"/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/_shared/testing.py",
 line 232, in inner
result = func(*args, **kwargs)
  File 
"/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/tests/test_rank.py",
 line 89, in check_all
rank.windowed_histogram(image, selem))
  File 
"/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
 line 986, in windowed_histogram
pixel_size=n_bins)
  File 
"/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
 line 90, in _apply_vector_per_pixel
pixel_size=pixel_size)
  File 
"/build/skimage-0.12.3/debian/tmp/usr/lib/python2.7/dist-packages/skimage/filters/rank/generic.py",
 line 53, in _handle_input
out = np.empty(image.shape+(pixel_size,), dtype=out_dtype)
TypeError: 'numpy.float64' object cannot be interpreted as an index

--
Ran 1383 tests in 181.697s


I'll leave it to the reader to consider lowering the severity of this
bug and risking that python-skimage (and possibly other packages) will
not reach the next stable release.  While python-skbio is a package of
minor importance python-skimage has several reverse depends and should
be saved.

Kind regards

  Andreas.

-- 
http://fam-tille.de