[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717619470 How about also undoing the tutorials? I guess their format will be wrong because you add indentation to text blocks. For the autotvm problem, I am okay to acce

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717619470 How about also undoing the tutorials? I guess their format will be wrong because you add indentation to text blocks. For the autotvm problem, I am okay to acce

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717588164 I added `__name__ == "__main__"` but the results are the same. Can you delete all autotvm related code and tutorials in this PR? We can investigate later.

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717588164 I added `__name__ == "__main__"` but the results are the same. Can you delete all autotvm related code and tutorials in this PR? We can investigate later.

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717588164 I added `__name__ == "__main__"` but the results are the same. Can you delete all autotvm related code and tutorials in this PR? We can investigate later.

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717588164 I added `__name__ == "__main__"` but the results are the same. Can you delete all autotvm related code and tutorials in this PR? We can investigate later.

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717550912 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717550912 Sorry, could you point me to the related lines? I only see you rename some variables in `python/tvm/autotvm/measure/local_executor.py` -

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717547728 Which PR introduced the changes that broke the tutorials? I don't find related changes in your PR. -

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717540915 What's your testing environment (python version, CPU, OS, ...)? Why is the tutorial broken? I can run it successfully. ---

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717540915 What's your testing environment? Why is the tutorial broken? I can run it successfully. This

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717506384 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717506384 I tested your autotvm implementation and found it is significantly slower. To do the test, you can run this tutorial (https://github.com/apache/incubator-t

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-27 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-717506384 I tested your autotvm implementation and found it is significantly slower. To do the test, you can run this tutorial (https://github.com/apache/incubator-t

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR solves the problem while not bringing any performance regression. But I am not sure about the autot

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. I believe this PR won't bring any performance regression. But I am not sure about the autotvm part. Could you delete

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716905146 The auto-scheduler part looks good to me. But I am not sure about the autotvm part. If you don't mind, I suggest deleting the autotvm part in this PR.

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716534539 Another concern is that I introduced a new kind of workload in this PR (#6710, https://github.com/apache/incubator-tvm/blob/5d93c61896acafe5d0b76b70615f2e2823cbf

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716534539 Another concern is that I introduced a new kind of workload in this PR (#6710, https://github.com/apache/incubator-tvm/blob/5d93c61896acafe5d0b76b70615f2e2823cbf

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-26 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-716534539 Another concern is that I introduced a new kind of workload in this PR (#6710) for relay integration. When using auto-scheduler in Relay, we don't serialize a

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-20 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-708845921 Autotvm uses a lot of python multiprocessing and I expect it will be much slower when using spawn. AutoTVM uses multiprocessing for feature extraction. So it ne

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-14 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-708845921 Autotvm uses a lot of python multiprocessing and I expect it will be much slower when using spawn. AutoTVM uses multiprocessing for feature extraction. So it ne

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-14 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-708845921 Autotvm uses a lot of python multiprocessing and I expect it will be much slower when using spawn. AutoTVM uses multiprocessing for feature extraction. So it ne

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-14 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-708845921 Autotvm uses a lot of python multiprocessing and I expect it will be much slower when using spawn. AutoTVM uses multiprocessing for feature extraction. So it ne

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-14 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-708845921 Autotvm uses a lot of python multiprocessing and I expect it will be much slower when using spawn. AutoTVM uses multiprocessing for feature extraction. So it ne

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-14 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-708845921 Autotvm uses a lot of python multiprocessing and I expect it will be much slower when using spawn. AutoTVM uses multiprocessing for feature extraction. So it ne

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-14 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-708845921 Autotvm uses a lot of python multiprocessing and I expect it will be much slower when using spawn. AutoTVM uses multiprocessing for feature extraction. So it ne

[GitHub] [incubator-tvm] merrymercy edited a comment on pull request #6671: [FIX,AUTOSCHEDULER] Fix auto_scheduler to run with multiprocessing's spawn start method

2020-10-13 Thread GitBox
merrymercy edited a comment on pull request #6671: URL: https://github.com/apache/incubator-tvm/pull/6671#issuecomment-708103769 This is an automated message from the Apache Git Service. To respond to the message, please log o