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 2021/05/06 20:15:34 UTC

[GitHub] [tvm] jroesch commented on pull request #7992: Remove minimum seed constraint on XGB Tuner

jroesch commented on pull request #7992:
URL: https://github.com/apache/tvm/pull/7992#issuecomment-833830299


   @comaniac even if we want a 500 default this should not be hardwired into the code, usually magic numbers are considered a bad coding practice, we should probably at least name this constant make this behavior configurable since 500 is arbitrary hyper parameter of the system 


-- 
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:
users@infra.apache.org