[GitHub] ankkhedia commented on issue #10424: Improved error message

2018-04-11 Thread GitBox
ankkhedia commented on issue #10424: Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#issuecomment-380551646 @piiswrong Fixed the warning issue. Could you please review it? This is an automated

[GitHub] ankkhedia commented on issue #10424: Improved error message

2018-04-10 Thread GitBox
ankkhedia commented on issue #10424: Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#issuecomment-380288366 @piiswrong I have removed redundant messages as discussed This is an automated

[GitHub] ankkhedia commented on issue #10424: Improved error message

2018-04-10 Thread GitBox
ankkhedia commented on issue #10424: Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#issuecomment-380282372 @piiswrong As discussed with you, I updated my PR to append message for the exact parameter(obtained from captured warnings) for which shape

[GitHub] ankkhedia commented on issue #10424: Improved error message

2018-04-09 Thread GitBox
ankkhedia commented on issue #10424: Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#issuecomment-379926490 @piiswrong I have made changes as suggested. Could you please take a look? This is

[GitHub] ankkhedia commented on issue #10424: Improved error message

2018-04-05 Thread GitBox
ankkhedia commented on issue #10424: Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#issuecomment-379093800 @piiswrong I have added error message for deferred inititialisation as mentioned by you in #9842 . Could you please review the changes