jcf94 commented on pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#issuecomment-657924465
> Does not have to change now, but let us change the use of ThreadPool to parallel_for abstraction. Does that means to just modify ThreadPool to ParallelFor now? Class renamed & added some comments on the member function. > @jcf94 and @merrymercy thanks for all the hard work! Can I request that we put another unresolved issue? In my opinion the written English parts i.e comments, explanations, etc could still use some improvement with both content and grammar and I would propose in general that we do some at least 1 or 2 rounds of full documentation polish (comments, examples, tests, tutorials, etc) before we officially release a feature (in this case when all of Ansor is landed in master). We tried to do this with Relay but I think we should continue to strive to do a better job with new features like this. Thanks! That would be of great help since I'm not a native speaker. The documentation does need to be polished. ---------------------------------------------------------------- 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]
