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/22 07:44:58 UTC

[GitHub] [tvm] shingjan opened a new pull request, #11098: [Auto Scheduler]add task name during printing table info

shingjan opened a new pull request, #11098:
URL: https://github.com/apache/tvm/pull/11098

   This PR intends to add task name within the `PrintTableInfo` callback function of `auto_scheduler` to improve the readability of related logs.
   
   cc: @zxybazh @junrushao1994  
   


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


[GitHub] [tvm] masahi merged pull request #11098: [Auto Scheduler]add task name during printing table info

Posted by GitBox <gi...@apache.org>.
masahi merged PR #11098:
URL: https://github.com/apache/tvm/pull/11098


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


[GitHub] [tvm] zxybazh commented on pull request #11098: [Auto Scheduler]add task name during printing table info

Posted by GitBox <gi...@apache.org>.
zxybazh commented on PR #11098:
URL: https://github.com/apache/tvm/pull/11098#issuecomment-1106736890

   Thanks for the contribution! Could you please fix the CI and also attached an example of the printed table in the introduction?


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


[GitHub] [tvm] comaniac commented on pull request #11098: [Auto Scheduler]add task name during printing table info

Posted by GitBox <gi...@apache.org>.
comaniac commented on PR #11098:
URL: https://github.com/apache/tvm/pull/11098#issuecomment-1106210971

   Overall LGTM. One miner suggestion is using "desc" instead of "name" to align with the attribute name. After all this is not an official task name but just a human readable hint.


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


[GitHub] [tvm] junrushao1994 commented on pull request #11098: [Auto Scheduler]add task name during printing table info

Posted by GitBox <gi...@apache.org>.
junrushao1994 commented on PR #11098:
URL: https://github.com/apache/tvm/pull/11098#issuecomment-1106199156

   Might need some tweak arounds aesthetically but sure I like this change :-)
   
   CC @Hzfengsy @merrymercy


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