tqchen edited a comment on issue #4882: [Relay][Topi] Hashtable support URL: https://github.com/apache/incubator-tvm/pull/4882#issuecomment-586359543 Thanks @lfengad for the PR. Given this PR involves a major technical decisions(introducing hashtable to the runtime), can you please open discuss RFC thread in the discuss forum first to discuss possible technical choices? In this way we can converge to the best design that fits our goals By quickly looking at the PR, I can see a few places that can be improved(which we can discuss in depth in the RFC). In particular, we don't have to introduce additional C APIs to the FFI, and should consolidate everything to use the Object instead. While relay's VM runtime already supports Object, it would be interesting to ask whether we want to extend some level of support to the graph RT
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
