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/04/29 18:32:00 UTC

[GitHub] [tvm] zxybazh commented on a diff in pull request #11157: [MetaSchedule] Logging Interface Unification

zxybazh commented on code in PR #11157:
URL: https://github.com/apache/tvm/pull/11157#discussion_r862047345


##########
python/tvm/meta_schedule/tune.py:
##########
@@ -17,7 +17,8 @@
 """User-facing Tuning API"""
 # pylint: disable=import-outside-toplevel
 import logging
-import os.path
+from pathlib import Path

Review Comment:
   I think it might be acceptable to use `exist_ok = True` in this case cause the logs folder could be reused.



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