marcoabreu commented on a change in pull request #11650: remove windows note 
for dataloader, fix deprecated warning in python3.7
URL: https://github.com/apache/incubator-mxnet/pull/11650#discussion_r202119634
 
 

 ##########
 File path: tests/python/unittest/test_gluon_data.py
 ##########
 @@ -205,24 +206,20 @@ def batchify(self, data):
                 nd.array(labels, dtype=labels.dtype, 
ctx=context.Context('cpu_shared', 0)),
                 nd.array(y_lens, ctx=context.Context('cpu_shared', 0)))
 
-
-    # This test is pointless on Windows because Windows doesn't fork
-    if platform.system() != 'Windows':
-        data = Dummy(True)
-        loader = DataLoader(data, batch_size=40, batchify_fn=data.batchify, 
num_workers=2)
-        for epoch in range(1):
-            for i, data in enumerate(loader):
-                if i % 100 == 0:
-                    print(data)
-                    print('{}:{}'.format(epoch, i))
-
-        data = Dummy(True)
-        loader = DataLoader(data, batch_size=40, 
batchify_fn=data.batchify_list, num_workers=2)
-        for epoch in range(1):
-            for i, data in enumerate(loader):
-                if i % 100 == 0:
-                    print(data)
-                    print('{}:{}'.format(epoch, i))
+    # sanity tests
+    # enable test on windows
+    data = Dummy(False)
 
 Review comment:
   I see, thanks for elaborating.
   
   Yeah, we had some disk space problems on our server. Everything should be 
fine now :)

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to