Wheest commented on pull request #7142: URL: https://github.com/apache/tvm/pull/7142#issuecomment-750293513
Thanks for the tip! I've added a test for `conv2d_nchw_int8`, with a structure much like you suggested. It fails in `main`, but is fixed in this PR. Through the test, I identified a silly mistake I had made in the workload definition in the original commit, now fixed. Does the style of this test meet the standards you expect? E.g. is the test being called as a nested function of `verify_conv2d_nchw_int8` the right place for this? If so, I will add a similar test for all of the ops touched by this PR. Otherwise, happy to take some suggestions on how to improve this test before I reproduce it elsewhere. ---------------------------------------------------------------- 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]
