zhreshold opened a new pull request #17841: Gluon data 2.0: c++ dataloader and 
built-in image/bbox transforms
URL: https://github.com/apache/incubator-mxnet/pull/17841
 
 
   ## Description ##
   This is the implementation for proposal #17269
   
   
   ### Changes ###
   The major components of this PR is:
   - c++ threaded dataloader with native support from the following 
datasets/batchify_functions
   - c++ native support for datasets covering `gluon.data.datasets` and 
`gluon.data.vision.datasets`
   - c++ batchify functions including `Stack`, `Pad`, `Groups`
   - Gluon version of transforms that can apply data augmentations to 
image/bboxes
   - Gluon version of data iterator substitutes of `mx.image.ImageIter` and 
`mx.image.ImageDetIter` for simplest interface for loading image classification 
and object detection datsets.
   
   ### Code Review ###
   Since this PR involves lots of backend and frontend changes and can not be 
easily divided into multiple PRs, so I would like to suggest modular code 
reviews:
   
   | Modules | notes | suggested reviewers | 
   | ------------| -------- | --------------------------- |
   | include/mxnet/c_api.h/cc, include/mxnet/io.h,  
python/mxnet/gluon/data/_internal.py,  python/mxnet/io/io.py | c api changes 
for io: datasets and batchify functions, with corresponding python frontends | 
@szha, @leezu |
   | python/mxnet/gluon/data/batchify.py(dataloader.py, dataset.py) | gluon 
data loader and batchify functions | @sxjscience |
   |  python/mxnet/gluon/contrib/data/vision/dataloader.py,  
python/mxnet/gluon/contrib/data/vision/transforms/* | substitutes for 
deprecated mx.image.ImageIter and mx.image.ImageDetIter and old augmenters | 
all committers |
   |  python/mxnet/gluon/nn/basic_layers.py | changes to `nn.sequential` and 
`nn.HybridSequential` to allow Sequential block to take more than 1 arguments | 
 all gluon contributors |
   |  python/mxnet/image/image.py | numpy compatitility change | @haojin2 |
   | src/imperative/* | Minor modification to cached op to allow operators to 
skip engine in imperative executions | @eric-haibin-lin |
   |  src/io/dataloader.cc(dataset.cc, batchify.cc), src/io/iter_sampler.cc | 
c++ implementations of gluon.data(gluon.data. dataset, gluon.data.batchify) | 
@leezu @sxjscience  |
   |  src/operator/image/* | operators to support many random image 
transformations | committers familiar with vision operators |
   
   Thank you for everyone who participate in code review and sorry about the 
size of this PR.
   
    
   
   

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

Reply via email to