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/08/18 08:43:41 UTC

[GitHub] [tvm] mkatanbaf commented on a diff in pull request #12479: [MicroTVM] expose project options in autotuning

mkatanbaf commented on code in PR #12479:
URL: https://github.com/apache/tvm/pull/12479#discussion_r948822307


##########
python/tvm/micro/testing/evaluation.py:
##########
@@ -32,20 +32,29 @@
 
 
 def tune_model(
-    platform, board, target, mod, params, num_trials, tuner_cls=tvm.autotvm.tuner.GATuner
+    platform,
+    board,
+    target,
+    mod,
+    params,
+    num_trials,
+    tuner_cls=tvm.autotvm.tuner.GATuner,
+    project_options=None,
 ):
     """Autotunes a model with microTVM and returns a StringIO with the tuning logs"""
     with tvm.transform.PassContext(opt_level=3, config={"tir.disable_vectorize": True}):
         tasks = tvm.autotvm.task.extract_from_program(mod["main"], {}, target)
     assert len(tasks) > 0
     assert isinstance(params, dict)
 
+    project_options = {
+        **(project_options if project_options is not None else {}),

Review Comment:
   Thanks for the comments. I weakly preferred not allowing to overwrite the project type, but in the interest of sticking with one behavior, I followed the implementation in _create_aot_session_ .



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