[GitHub] [incubator-mxnet] zhreshold commented on issue #17841: Gluon data 2.0: c++ dataloader and built-in image/bbox transforms

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

zhreshold commented on issue #17841: Gluon data 2.0: c++ dataloader and built-in image/bbox transforms
URL: https://github.com/apache/incubator-mxnet/pull/17841#issuecomment-614330741
 
 
   > Let's use constructor to initialize the classes instead of requiring users to call Init after construction?
   
   @leezu Sorry I missed this comment. If you look at the creator in c_api used for list and create the existing mxnet iterators: https://github.com/apache/incubator-mxnet/blob/master/src/c_api/c_api.cc#L1859, you will find that the registry is used to hold all uninitialized iterators that has been registered to io module: https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/io/io.py#L984
   
   So if I follow the current registeration patter in python module, I have to follow the empty constructor + init function. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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