ilastik/ilastik — pull request #858 by stuarteberg (2014-03-15 19:35:24+00:00)
Title: Replace counting label operator

@buotex, I am working to eliminate `OpBlockedSparseLabelArray` and `OpSparseLabelArray` from the ilastik code base, since they can be replaced with faster alternatives. (Eventually we will probably use something from the `blockedarray` repo for labeling, but for now I'm using pure-python alternatives.)

Anyway, I hacked a bit on the counting top-level operator to replace the labeling operator. Instead of `OpBlockedSparseLabelArray`, it now uses `OpDenseLabelArray` (which was originally written for the Carving workflow. It is faster because it doesn't store a sparse representation of the labels -- it just stores the raw array data.

These changes seem to work just fine in my testing (and all counting unit/regression tests still pass...). Still, I'd like it if you could just give it a quick try and hit "merge" on this PR if you don't see any issues. Thanks!