merrymercy edited a comment on pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#issuecomment-657068540
Thank all the reviewers! Most comments from the reviewers are addressed. We summarize the unresolved comments here. They will be addressed by follow-up PRs. We want to merge this PR first to accelerate the upstream process. - The design of `python/tvm/auto_schedule/auto_scheduler.py::auto_schedule`. This is the most important user interface to use the auto-scheduler. We will send an RFC to discuss this. - Move `src/auto_schedule/utils.h::ThreadPool` to `src/support/parallle_for.h` (create a new file) - Consolidate the usage of `python/tvm/auto_schedule/utils.py::get_const_int`, `python/tvm/auto_schedule/utils.py::get_const_tuple` in the whole TVM code base. @jroesch @jwfromm @yangjunpro @MarisaKirisame @junrushao1994 @FrozenGene Please take another look and approve. ---------------------------------------------------------------- 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]
