comaniac opened a new pull request #7389: URL: https://github.com/apache/tvm/pull/7389
When tuning workload using AutoTVM, we may encounter an exception saying `< not supported between instances of function and function`. The reason is we currently put RPC requests in the format of `(-priority, time.time(), callback)`. When two requests with the same `priority` and `time.time()`, then heap pop throws the error of trying to compare two callbacks. Although it seems rare to have the same timestamp for two requests, one of my colleagues encounters an exception repeatedly when tuning a task using AutoTVM on V100 GPU. As a result, I try to use a counter to replace the timestamp so that we guarantee there's no duplication. Not sure if we need thread locking here to make sure thread safe, tho. cc @tqchen @FrozenGene ---------------------------------------------------------------- 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]
