[GitHub] caiqi opened a new issue #12619: Too large max depth value in _recursive_fork_recordio

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

caiqi opened a new issue #12619: Too large max depth value in _recursive_fork_recordio
URL: https://github.com/apache/incubator-mxnet/issues/12619
 
 
   It seems that 1000 is too large for _recursive_fork_recordio in 
   https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/gluon/data/dataloader.py#L178
   Even if len(obj.__dict__.items()) > 2, this function will be called by more than 2 ** 1000 times. 
   
   The following code in https://github.com/dmlc/gluon-cv/blob/master/scripts/detection/ssd/train_ssd.py#L96  in gluon-cv will cause `RecursionError: maximum recursion depth exceeded in comparison` error on windows 10 with the latest build. I found that the reason is that there will be a HybridSequential object in the dataset object and the HybridSequential contains many children. This function is brought in commit #12554 . Is it ok to jump out of this function when obj is not an instance of mx.gluon.data.dataset.Dataset?

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