u99127 commented on issue #5355: URL: https://github.com/apache/incubator-tvm/pull/5355#issuecomment-616392562
> > @siju-samuel is correct. If we do like this pr, when we `import tvm`, the users will be required to install tflite package, which is not we want to see. > > I missed that when I did the PR and now notice this to be the case in all the frontends, so I'll fix that. > > I take it also that it is desirable to reduce the duplication and it should be possible to find another suitable location within the same file. The number of imports now reduces from about 120 to 53. Now only restricted to tflite.Operator, please help review and merge . Ramana ---------------------------------------------------------------- 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