[GitHub] [incubator-tvm] tqchen edited a comment on issue #4882: [Relay][Topi] Hashtable support

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services