[GitHub] [incubator-mxnet] samskalicky commented on pull request #16131: Fix for duplicate subgraph inputs/outputs

[GitBox <gi...@apache.org>]

samskalicky commented on pull request #16131:
URL: https://github.com/apache/incubator-mxnet/pull/16131#issuecomment-690811656


   > @mseth10 looks like we made an aux an arg somehow, thats why the MKLDNN tests are failing:
   > 
   > in_args_map	arg_names	arg_names
   > data	data	data
   > conv1_weight	conv1_weight	conv1_weight
   > bn1_gamma	bn1_gamma	bn1_gamma
   > bn1_beta	bn1_beta	bn1_beta
   > conv2_weight	conv2_weight	conv2_weight
   > bn2_gamma	bn2_gamma	bn2_gamma
   > bn2_beta	bn2_beta	bn2_beta
   > **bn2_moving_mean**
   > So this is why the error says
   > 
   > ```
   > MXNetError: Check failed: arg_names.size() == in_args_map.size() (8 vs. 7) : 
   > ```
   > 
   > Theres one extra name in `arg_names` for `bn2_moving_mean`
   
   For the record @mseth10 i finally found the issue that caused this "making an aux an arg" problem: https://github.com/apache/incubator-mxnet/pull/19112#issuecomment-690810445


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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