[GitHub] [incubator-tvm] jcf94 commented on pull request #6184: [Ansor][AutoTVM v2.0] Phase 2: Basic CPU Sketch Search Policy [WIP]

2020-08-05 Thread GitBox 


jcf94 commented on pull request #6184:
URL: https://github.com/apache/incubator-tvm/pull/6184#issuecomment-669172637


   This PR shares the base class with cost models, will rebase the code after 
#6187 has been merged.
   The other parts of code is ready for review.
   cc @merrymercy @comaniac @FrozenGene @junrushao1994 



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

[GitHub] [incubator-tvm] jcf94 commented on pull request #6184: [Ansor][AutoTVM v2.0] Phase 2: Basic CPU Sketch Search Policy [WIP]

2020-08-02 Thread GitBox 


jcf94 commented on pull request #6184:
URL: https://github.com/apache/incubator-tvm/pull/6184#issuecomment-667757439


   > @jcf94 I sent a separate PR for the base class for cost models (#6187).
   > Why do we need to rely on the random seed here?
   
   Thanks!
   The random seed issue does not block anything. I addressed it here is just 
to make sure we're able to fully reproduce a search process. The current UT 
generates different states in each run.



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

[GitHub] [incubator-tvm] jcf94 commented on pull request #6184: [Ansor][AutoTVM v2.0] Phase 2: Basic CPU Sketch Search Policy [WIP]

2020-08-01 Thread GitBox 


jcf94 commented on pull request #6184:
URL: https://github.com/apache/incubator-tvm/pull/6184#issuecomment-667514867


   @merrymercy I realized that a basic class of CostModel is also needed in 
this PR. We can sync that part first when you start to work on the cost model 
PR.



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