soeque1 edited a comment on issue #12796: Add embedding to print_summary URL: https://github.com/apache/incubator-mxnet/pull/12796#issuecomment-429225606 @marcoabreu The test_print_summary() just print the output that have a lot of model information. It seems to be no problem. Another test may be required to verify the number of each layer's parameters. It is related to the definition that print_layer_summary() is the inner definition of print_summary(). But, Wouldn't you rather separate it from this issue(#12796)?
---------------------------------------------------------------- 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
