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 2022/09/05 08:10:27 UTC

[GitHub] [tvm] echuraev commented on pull request #12545: [AutoTVM] Introducing multi_filter into ConfigSpace autotvm.

echuraev commented on PR #12545:
URL: https://github.com/apache/tvm/pull/12545#issuecomment-1236674985

   Quickly looked into this PR. I don't really like that now we need to call tsk.config_space.filtered_length instead of len(tsk.config_space). I think that len(smth) is more intuitive. But I didn't dive deeply into the details. I'll do it later. I think that @elvin-n can provide more useful and detailed review.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org