quic-sanirudh commented on PR #15928: URL: https://github.com/apache/tvm/pull/15928#issuecomment-1763964991
> LGTM. Do you plan to change access names in unit tests? Thanks for the review, and yes, I'll update the tests and push it today. Initially I thought it might introduce a lot of changes that hurts git history un-necessarily, but now I think there could be some advantage to doing it (especially since tests are seen as an initial source of example usage of an API). -- 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]
