You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2018/12/07 11:56:47 UTC

[GitHub] BeyonderXX opened a new pull request #13575: Fix the bug of BidirectionalCell

BeyonderXX opened a new pull request #13575: Fix the bug of BidirectionalCell
URL: https://github.com/apache/incubator-mxnet/pull/13575
 
 
   
   ## Description ##
   I did hybridize( ) and pass "valid_length" to the unroll( ) function of BidirectionalCell, then returned AssertionError in line 79. Because symbol.split( ) return a symbol but not a symbol list. Result in the length of inputs dont equal parameter "length"  when call unroll( )  to compute r_outputs and r_states.
   
   
   ### Changes ###
   - [line 1044] change _as_list( ) to list( )
   - [line 1066] change _as_list( ) to list( )
   
   ## Comments ##
   - Symbol support for converting to lists through list( ) API, so that use list( ) function insted of _as_list( ).
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services