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: [email protected] With regards, Apache Git Services
