[GitHub] safrooze opened a new pull request #13390: Onnx multi output

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

safrooze opened a new pull request #13390: Onnx multi output
URL: https://github.com/apache/incubator-mxnet/pull/13390
 
 
   ## Description ##
   ONNX export does not create a correct graph when there are multiple outputs. This PR fixes this bug.
   
   ## Checklist ##
   ### Essentials ###
   Please feel free to remove inapplicable items for your PR.
   - [x] Changes are complete (i.e. I finished coding on this PR)
   - [x] All changes have test coverage:
   - Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
   - [x] Code is well-documented: 
   - [x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change
   
   ### Changes ###
   - Fixed ONNX export to handle graphs with multiple outputs.
   - Added unit-test for ONNX export and output to test both single output and multi-output graphs
   
   ## Comments ##
   Before this change, export_onnx would treat the graph as a sequential graph with output as the last node in the sequence. This change fixes this behavior by using sym.list_outputs() to record all output nodes.
   

----------------------------------------------------------------
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