You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2019/12/05 08:01:41 UTC

[GitHub] [incubator-tvm] masahi commented on issue #4009: [RFC] 3D model support

masahi commented on issue #4009: [RFC] 3D model support
URL: https://github.com/apache/incubator-tvm/issues/4009#issuecomment-562016662
 
 
   @optima2005 yes, I prefer a unified implementation. But that can potentially affect existing users who are using 2D only ops. Most people dont care about 3D, so if we generalize some ops for 3D and in the process introduce perf regression or other bugs to exising 2D users, they will get upset. 
   
   I think one of the requiments for adding 3D support is to introduce absolutely ZERO impact to existing 2d users. In that sense, I think creating seperate 3D implementation, which will involve some code duplication, seems not ideal but a safer bet. Can you comment? @tqchen @yzhliu @soiferj @vinx13 
   
   

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