piyushghai edited a comment on issue #12783: Name conflict when serializing 
HybridSequentialRNNCell
URL: 
https://github.com/apache/incubator-mxnet/issues/12783#issuecomment-428762224
 
 
   @lostella Here are my findings on the issue : 
   
   The symbol names are not getting saved properly in the symbol.json file 
generated. 
   More specifically, with the unroll sequence length generates/replicates the 
same LSTM layer 'n' times, where n is the unroll seq length. 
   Now if you closely examine the layer name in error : 
```myblock0_lstm0__plus0_output``` it has a double``` '_'``` in between 
indicating something is amiss here. 
   On further investigation, I found that a time stamp suffix : ```t0, t1``` 
etc indicating the unroll sequence number is missing here. To verify a quick 
fix, I opened up the symbol.json and manually added the time prefixes on places 
which were complaining about the error in imports method. 
   I also had to fix the suffix issue in activation layers with name prefix as 
: ```myblock0_lstm<layer_number>_activation<time-stamp>``` to have correct 
values for timestamp.
   eg : ```myblock0_lstm2_activation0```
   After fixing the symbol.json file, the imports statement was working fine.
   
   I will now investigate the root cause of this issue to fix it in code. 
   
   Attached are gists of my working .ipynb notebook, and the corrected 
symbol.json file. 
   
   https://gist.github.com/piyushghai/ad18f1290ec05d96ef5e9631474ae553
   

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to