zxybazh commented on a change in pull request #10079:
URL: https://github.com/apache/tvm/pull/10079#discussion_r819044834
##########
File path: src/meta_schedule/task_scheduler/task_scheduler.cc
##########
@@ -124,7 +124,7 @@ void TaskSchedulerNode::Tune() {
int running_tasks = tasks.size();
for (int task_id; (task_id = NextTaskId()) != -1;) {
- LOG(INFO) << "Scheduler picks Task #" << task_id << ": " <<
tasks[task_id]->task_name;
+ LOG(INFO) << "Scheduler picks Task #" << task_id + 1 << ": " <<
tasks[task_id]->task_name;
Review comment:
Hi, thanks for pointing that out. The reason for this change is
previously we have `task 0/18` to `task 17/18` which seems confusing to me. I
intend to make sure the log is consistent but I might missed a couple places.
After second thoughts I think it might be better to keep the `task_id` the same
way it is so that it would be easier for debugging later. See #10478.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
