CircleSpin commented on a change in pull request #7792:
URL: https://github.com/apache/tvm/pull/7792#discussion_r608117595



##########
File path: python/tvm/driver/tvmc/autotuner.py
##########
@@ -138,7 +138,11 @@ def add_tune_parser(subparsers):
         "Autoscheduler options",
         "Autoscheduler options, used when --enabled-auto-scheduler is 
provided",
     )
-
+    auto_scheduler_group = parser.add_argument(
+        "--enable-hardware-params",
+        help="enable hardware specific controls such as cores, threads, memory 
etc.",

Review comment:
       Sorry! I should have used store_true. I think that made everything 
confusing. My goal was to use the default auto_schedule parameters if the user 
didn't specify anything. Hardware parameters can still be specified in the 
command line as in the main branch. 
   
   Perhaps we can push this PR where advanced users can still use command line 
to provide specific hardware arguments, and consider doing the JSON extension 
later? 




-- 
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:
[email protected]


Reply via email to