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:

With regards,
Apache Git Services

Reply via email to