You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/07/24 02:16:40 UTC

[GitHub] [incubator-tvm] merrymercy opened a new issue #6133: Ansor Stabalization Tracking

merrymercy opened a new issue #6133:
URL: https://github.com/apache/incubator-tvm/issues/6133


   This issue is used to track all unresolved todo items in Ansor during upstream.
   
   ### Motivation and Plan
   Ansor has more than 13K lines of code in our internal codebase.
   To upstream the huge code base of Ansor, we divide Ansor into several (10+) small PRs to make the review process easier.
   
   However, addressing every little detail in these small PRs makes the upstream process slow and painful.
   The refactor/renaming happens in one PR will affect later PRs and bring extra efforts to the latter PRs because we have to maintain two codebases at the same time. (i.e. tvm master and our internal codebase)
   
   Therefore, the plan is to merge these small PRs quickly and then do a round of polish before the release.
   The polish phase will happen after we got most of the code merged. So we do not have to maintain two codebases and only need to do refactor/renaming in a single place.
   The polish phase will mainly focus on API design and documentation.
   
   This plan does not mean the quality of the small PRs is low. Actually, all these small PRs are carefully reviewed by multiple people and most of the comments are addressed. The first PR of Ansor got 436 comments, making it one of the most commented PR, if not the most, in the TVM history.
   
   
   ### Todo items
   - 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.
   - 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.
   - Improve the hash of `LoopState` and `TransformStep`. Try to implement it with `SHashReduce`, `SEqualReduce`
   - Improve the record serialization format. Do not use only a hash key as the workload key. Try to use a relay subgraph as the hash key.
   
   
   


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

[GitHub] [incubator-tvm] tqchen commented on issue #6133: Ansor Stabilization Todo Items Tracking

Posted by GitBox <gi...@apache.org>.
tqchen commented on issue #6133:
URL: https://github.com/apache/incubator-tvm/issues/6133#issuecomment-663867839


   cc @jroesch @jwfromm 


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

[GitHub] [incubator-tvm] tqchen edited a comment on issue #6133: Ansor Stabilization Todo Items Tracking

Posted by GitBox <gi...@apache.org>.
tqchen edited a comment on issue #6133:
URL: https://github.com/apache/incubator-tvm/issues/6133#issuecomment-663867839


   cc @jroesch @jwfromm @yzhliu @ajtulloch 


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