comaniac commented on pull request #8862: URL: https://github.com/apache/tvm/pull/8862#issuecomment-906812284
Thanks for bringing this up. We have considered this before for the same reason. Another benefit of not using hash is we can recover the task simply with workload_key. Now we cannot do that because we have no idea how to construct the DAG with the hash. However, the DAG could be very large, so the log file size is indeed a concern. That's why the workload key still looks like this format. Customize hash function may be another direction to go, but it also means we have to make sure the customized function is serialized and passed correctly in all use cases (e.g., tuning with RPC, deployment, etc); otherwise it's like you get an encoded file without getting a private key 😂 -- 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]
