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/09/14 22:57:54 UTC

[GitHub] mdespriee opened a new pull request #12570: [WIP] review require() usages to add meaningful messages.

mdespriee opened a new pull request #12570: [WIP] review require() usages to add meaningful messages.
URL: https://github.com/apache/incubator-mxnet/pull/12570
 
 
   ## Description ##
   The motivation of this PR is to add & improve error messages raised by failed requirements.
   In relation to https://github.com/apache/incubator-mxnet/issues/12565.
   
   ## Checklist ##
   ### Essentials ###
   Please feel free to remove inapplicable items for your PR.
   - [ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant [JIRA issue](https://issues.apache.org/jira/projects/MXNET/issues) created (except PRs with tiny changes)
   - [ ] Changes are complete (i.e. I finished coding on this PR)
   
   ### Changes ###
   No feature change.
   
   ## Comment ##
   Unfortunately, i'm not sure to really improve the error messages regarding shape inference. Real improvement would come from backend error messages. Any suggestions ?
   
   

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