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 2019/01/24 22:07:02 UTC

[GitHub] lanking520 edited a comment on issue #13982: Unnecessary requirement

lanking520 edited a comment on issue #13982: Unnecessary requirement
URL: https://github.com/apache/incubator-mxnet/issues/13982#issuecomment-457375770
 
 
   @yx3zhang Thanks for reporting this issue. This requirement method introduced in 1.2 and improved in 1.3. We plan to make the layout as a requried field for DataDesc. Due to the backward compatibility, we add a `undefined` placeholder here. Please feel free to pass the `UNDEFINED` if you don't want this be in the model. Please let me know if you are facing more issues like that and bring some code here that can reproduce your issue. I will take a look and see how I can help.
   
   Some of our new APIs such as Predictor API (in 1.2) required this field to be set so that we add a required in there.

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