Mailman 3 October 2012 - scikit-image

each_channel decorator
by Tony Yu Oct. 16, 2012

Oct. 16, 2012

After my failed @grayscale decorator suggestion (sorry again Johannes for all the unnecessary work this caused you), I was thinking it might be nice to add an @each_channel decorator. The basic idea is that the decorator would check if the input image is RGB, and applies the grayscale function to each channel of an RGB image. Cheers, -Tony

1 0

Scikits project homepage
by Johannes Schönberger Oct. 13, 2012

Oct. 13, 2012

I just came across http://scikits.appspot.com/scikits and currently scikit-image is not listed here.

2 1

Re: v0.8 release
by Johannes Schönberger Oct. 12, 2012

Oct. 12, 2012

I would release a 0.7.1 to address the renaming and use 0.8 for new features. So, make a feature freeze at the moment - until 0.7.1 is released. Am Dienstag, 9. Oktober 2012 09:27:22 UTC+2 schrieb Stefan van der Walt: > > On Mon, Oct 8, 2012 at 7:20 PM, Tony Yu <tsy...(a)gmail.com <javascript:>> > wrote: > > That said, I'm inclined to merge PR #351 and PR #360. > > PR #351 looks good to me. > > > Are you planning to release both 0.7.1 *and* 0.8? Is there any reason > why we > > shouldn't just point users to 0.8? > > Some people will still try to install "scikits-image", which currently > gives 0.7--broken on OSX. As such, it would be nice to at least have > a working 0.7. > > Stéfan >

2 1

Re: Cython fused types
by Johannes Schönberger Oct. 12, 2012

Oct. 12, 2012

Are those binaries really that huge? I just tested this locally and the binaries were larger, but imho not too big. The question is who actually cares about a few extra KB disk space taken by a binary? Am Montag, 8. Oktober 2012 20:47:45 UTC+2 schrieb Stefan van der Walt: > > On Mon, Oct 8, 2012 at 10:38 AM, Mathieu Blondel <mat...(a)mblondel.org<javascript:>> > wrote: > > Fused types were actually introduced in v0.16 > > (http://wiki.cython.org/ReleaseNotes-0.16) so v0.17 is the second > release > > supporting them. > > The problem I see with fused types is that they generate very large > binaries. As far as I know, this is also why they are not being used > in SciPy currently. > > Stéfan >

2 1

Re: v0.8 release
by Stéfan van der Walt Oct. 9, 2012

Oct. 9, 2012

On Mon, Oct 8, 2012 at 7:20 PM, Tony Yu <tsyu80(a)gmail.com> wrote: > That said, I'm inclined to merge PR #351 and PR #360. PR #351 looks good to me. > Are you planning to release both 0.7.1 *and* 0.8? Is there any reason why we > shouldn't just point users to 0.8? Some people will still try to install "scikits-image", which currently gives 0.7--broken on OSX. As such, it would be nice to at least have a working 0.7. Stéfan

1 0

Re: v0.8 release
by Tony Yu Oct. 9, 2012

Oct. 9, 2012

On Mon, Oct 8, 2012 at 8:04 PM, Stéfan van der Walt <stefan(a)sun.ac.za>wrote: > Hello, everyone > > With all the renaming of the past week, it would be good if we could > make a new release. Are there any outstanding PRs we should consider > including? > > I may also push 0.7.1 as a bug-fix release for building on OSX. > > Stéfan > > -- > Yeah, the renaming definitely wreaked havoc on the 0.7.0 release. Let's try to avoid another rename ;) There are definitely a few good PRs waiting to be reviewed, but I'm inclined to wait until they've had some thorough vetting. (We could really use some help with review, if anyone out there has some free time---especially on some of the longer PRs.) That said, I'm inclined to merge PR #351<https://github.com/scikit-image/scikit-image/pull/351>and PR #360 <https://github.com/scikit-image/scikit-image/pull/360>. Are you planning to release both 0.7.1 *and* 0.8? Is there any reason why we shouldn't just point users to 0.8? -Tony

1 0

v0.8 release
by Stéfan van der Walt Oct. 9, 2012

Oct. 9, 2012

Hello, everyone With all the renaming of the past week, it would be good if we could make a new release. Are there any outstanding PRs we should consider including? I may also push 0.7.1 as a bug-fix release for building on OSX. Stéfan

1 0

Re: Cython fused types
by Mathieu Blondel Oct. 8, 2012

Oct. 8, 2012

On Tue, Oct 9, 2012 at 3:17 AM, Tony Yu <tsyu80(a)gmail.com> wrote: > > I actually tried to bump up the Cython requirement to 0.16 (see > https://groups.google.com/forum/?fromgroups=#!topic/scikit-image/Fzdtoo8AUZ0) > about a month ago, but that turned out to be **way** too soon. > > For example, the current Ubuntu, 12.04, ships with Cython 0.15. I guess > Ubuntu 12.10 will be released soon, and that release includes Cython 0.16. > That said, I'm not sure if targeting the most recent Ubuntu is what we want > to do (since it will be too fast for some people). > > Andreas or Mathieu: Does scikit-learn have a policy for upgrading > dependencies (in particular Cython)? > In scikit-learn, we've decided to use Cython 0.17 but we include generated .c files in the git repository so the dependency is only for developers who want to regenerate a particular .c file from the corresponding .pyx file... A downside is that this generates large and hard to read diffs but we recently merged a .gitattributes file that should alleviate this problem: https://github.com/scikit-learn/scikit-learn/blob/master/.gitattributes Mathieu

1 0

Re: Cython fused types
by Stéfan van der Walt Oct. 8, 2012

Oct. 8, 2012

On Mon, Oct 8, 2012 at 10:38 AM, Mathieu Blondel <mathieu(a)mblondel.org> wrote: > Fused types were actually introduced in v0.16 > (http://wiki.cython.org/ReleaseNotes-0.16) so v0.17 is the second release > supporting them. The problem I see with fused types is that they generate very large binaries. As far as I know, this is also why they are not being used in SciPy currently. Stéfan

1 0

Re: Cython fused types
by Tony Yu Oct. 8, 2012

Oct. 8, 2012

On Mon, Oct 8, 2012 at 1:38 PM, Mathieu Blondel <mathieu(a)mblondel.org>wrote: > > > On Tue, Oct 9, 2012 at 2:11 AM, Johannes Schönberger < > hannesschoenberger(a)gmail.com> wrote: > >> I'm not sure if everyone has already noticed, that there is support for >> "fused types" in the recently released Cython 0.17 ( >> http://docs.cython.org/src/userguide/fusedtypes.html). >> > > Fused types were actually introduced in v0.16 ( > http://wiki.cython.org/ReleaseNotes-0.16) so v0.17 is the second release > supporting them. > > Mathieu > > -- > I actually tried to bump up the Cython requirement to 0.16 (see https://groups.google.com/forum/?fromgroups=#!topic/scikit-image/Fzdtoo8AUZ0) about a month ago, but that turned out to be **way** too soon. For example, the current Ubuntu, 12.04, ships with Cython 0.15. I guess Ubuntu 12.10 will be released soon, and that release includes Cython 0.16. That said, I'm not sure if targeting the most recent Ubuntu is what we want to do (since it will be too fast for some people). Andreas or Mathieu: Does scikit-learn have a policy for upgrading dependencies (in particular Cython)? I'd lean toward more conservative versions of numpy, but more recent versions of Cython. I'm not sure how that would actually translate into hard numbers. Maybe numpy matches latest Ubuntu LTS, cython matches latest Ubuntu? (I don't run Ubuntu, btw, I just find it a convenient reference.) Or maybe some predetermined time delay after those releases? Best, -Tony

1 0