[GitHub] szha commented on issue #11482: make gluon rnn layers hybrid blocks

2018-08-02 Thread GitBox
szha commented on issue #11482: make gluon rnn layers hybrid blocks URL: https://github.com/apache/incubator-mxnet/pull/11482#issuecomment-410089794 @Roshrini this PR uncovers a weird bug which should probably be addressed before releasing.

[GitHub] szha commented on issue #11482: make gluon rnn layers hybrid blocks

2018-06-29 Thread GitBox
szha commented on issue #11482: make gluon rnn layers hybrid blocks URL: https://github.com/apache/incubator-mxnet/pull/11482#issuecomment-401501912 I can override forward but it would be pretty much equivalent. The reason I cannot do this in hybrid_forward is that when given partial

[GitHub] szha commented on issue #11482: make gluon rnn layers hybrid blocks

2018-06-29 Thread GitBox
szha commented on issue #11482: make gluon rnn layers hybrid blocks URL: https://github.com/apache/incubator-mxnet/pull/11482#issuecomment-401484871 @piiswrong turns out I cannot do that only in infer_shape. The reason is that sometimes the block is used as a child block of other blocks,