Lunderberg commented on pull request #10515: URL: https://github.com/apache/tvm/pull/10515#issuecomment-1075472984
That's correct. I initially added this test to validate copying across a few orders of magnitude, and a fixed input size would work just as well. If we want to have the test be randomized per run, we could remove the call to `np.random.seed`. I'm fine with it either way, or switching to a fixed array. Most of the other unit tests don't run into the xdist incompatibility, because they didn't change the name of the test being run. It only became an issue here because the seed value was included in the parametrized test name. I've found it really useful to be able to use xdist to speed up local test times. Other than some false positives from running out of GPU memory, most of the other tests are xdist-friendly. -- 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]
