Cython version to build master
Hi everybody. I haven't kept track of all of the many PRs this last weeks, but I noticed today that I can't build master. There is an error in "skimage/morphology/_greyreconstruct.pyx". I guess that is because I use Cython 0.15.1. Is that correct? Do we only support Cython 0.16 now? Cheers, Andy
I just saw the commit with the change in the docs. Is this because of the memory views? Can someone point me to the discussion? Not being able to build scikits-image on current Ubuntu seems a bit annoying to me. On 08/25/2012 07:00 PM, Andreas Mueller wrote:
Hi everybody. I haven't kept track of all of the many PRs this last weeks, but I noticed today that I can't build master. There is an error in "skimage/morphology/_greyreconstruct.pyx". I guess that is because I use Cython 0.15.1. Is that correct? Do we only support Cython 0.16 now?
Cheers, Andy
On Sat, Aug 25, 2012 at 1:19 PM, Andreas Mueller <amueller@ais.uni-bonn.de>wrote:
I just saw the commit with the change in the docs. Is this because of the memory views? Can someone point me to the discussion? Not being able to build scikits-image on current Ubuntu seems a bit annoying to me.
Argh. I didn't realize this. I think it would probably be a good policy if the current Ubuntu version sets a *maximum* version on our dependencies. -T
On 08/25/2012 07:00 PM, Andreas Mueller wrote:
Hi everybody. I haven't kept track of all of the many PRs this last weeks, but I noticed today that I can't build master. There is an error in "skimage/morphology/_** greyreconstruct.pyx". I guess that is because I use Cython 0.15.1. Is that correct? Do we only support Cython 0.16 now?
Cheers, Andy
On Sat, Aug 25, 2012 at 1:00 PM, Andreas Mueller <amueller@ais.uni-bonn.de>wrote:
Hi everybody. I haven't kept track of all of the many PRs this last weeks, but I noticed today that I can't build master. There is an error in "skimage/morphology/_** greyreconstruct.pyx". I guess that is because I use Cython 0.15.1. Is that correct? Do we only support Cython 0.16 now?
Cheers, Andy
Hi Andy, This is my fault. I should have asked the list first: I bumped up the Cython version to use typed memoryviews<http://docs.cython.org/src/userguide/memoryviews.html> . My rationale was that we should be pretty conservative with older, more-entrenched packages like numpy and matplotlib, but less conservative with something that's rapidly evolving, like Cython. I'm fine with converting morphological reconstruction to support Cython 0.15 if there's push back. Cheers, -Tony
participants (2)
-
Andreas Mueller
-
Tony Yu