comaniac commented on pull request #7779: URL: https://github.com/apache/tvm/pull/7779#issuecomment-811548804
> @comaniac, if you look at the full function, its already handling `hostname=None` and requiring that `device` is set. The change to the arguments here just more closely matches the actual behavior. You're right. Yeah then it makes sense to adjust the arguments to align the actual behavior. Then please change the order accordingly and update the type in docstring. -- 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]
