Tomer Kaftan

Results 33 comments of Tomer Kaftan

Should we remove this from release 0.1 while we wait on https://github.com/scalanlp/breeze/issues/404#issuecomment-100368432

@etrain how would you guys feel about an interface kind of like this? ``` scala abstract class NumericTransformer[T : BreezeNumeric] extends Transformer[DenseVector[T], DenseVector[T]] { def apply(in: DenseVector[T]): DenseVector[T] = {...

Especially because it's easier to translate Matlab code to work on DenseMatrix than DenseVector

Fair enough. With that in mind how do you feel about the above interfaces?

I guess you would prefer requiring the single vector impl to be implemented and having a default matrix impl instead of the above?

It is, but there are enough cases where we want to do an "on matrix" / batch operation on what's otherwise a normal NumericTransformer that I think this is a...

There's that case and there's all the cases in the image pipelines, or e.g. LDA on the frames in the TIMIT pipeline

So, I actually looked at Mixins for this previously. To be able to mix in something you have to construct it with the "new" command, so you can't use a...

We don't believe the reshape layer supports masking at this time, though it could be added. We are happy to accept contributions that add it as a feature (Separately, we...

After talking with @dcrankshaw: We shouldn't fail a request that's sent to the wrong partition, we should forward it to the correct partition & log that it needed to do...