leezu commented on a change in pull request #17841: Gluon data 2.0: c++
dataloader and built-in image/bbox transforms
URL: https://github.com/apache/incubator-mxnet/pull/17841#discussion_r409188740
##########
File path: include/mxnet/io.h
##########
@@ -113,5 +120,105 @@ struct DataIteratorReg
*/
#define MXNET_REGISTER_IO_ITER(name) \
DMLC_REGISTRY_REGISTER(::mxnet::DataIteratorReg, DataIteratorReg, name)
+
+/*!
+ * \brief A random accessable dataset which provides GetLen() and GetItem().
+ * Unlike DataIter, it's a static lookup storage which is friendly to random
access.
+ * The dataset itself should NOT contain data processing, which should be
applied during
+ * data augmentation or transformation processes.
+ */
+class Dataset {
+ public:
+ /*!
+ * \brief Initialize the Operator by setting the parameters
+ * This function need to be called before all other functions.
+ * \param kwargs the keyword arguments parameters
+ */
+ virtual void Init(const std::vector<std::pair<std::string, std::string> >&
kwargs) = 0;
Review comment:
This is still outstanding. Do you have a plan to fix the API design issue?
----------------------------------------------------------------
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
