[GitHub] [tvm] zhiics commented on pull request #7149: Sparse segment sum op

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

zhiics commented on pull request #7149:
URL: https://github.com/apache/tvm/pull/7149#issuecomment-752310779


   @tkonolige Thanks for suggestions. I agree that it might be worth to discuss if a `sparse` namespace is needed although there are already some sparse operators in the code base without using the new namespace.
   
   However, I personally don't think we need to over complicate the process of implementing operators (e.g. comparing across all frameworks) as well, which is also not the process we have taken to implement other operators.
   
   In addition, it may be impractical and/or inefficient to use existing operators to implement some TF/PT ops IMHO. Particularly, there are not that many sparse op in relay to compose a synthetic TF op. 
   
   @codeislife99 could you summarize the discussion so that we can have agreements on all points to move forward?
   


----------------------------------------------------------------
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