comaniac commented on pull request #7823: URL: https://github.com/apache/tvm/pull/7823#issuecomment-818213495
> @comaniac, the OOP approach was something we discussed and are on the fence about implementing. The reason we went with this functional API is because we want this API to be catered to new users trying to understand TVM. These users will often not be familiar with ML deployment and breaking the process down into 4 concrete steps (load, tune, compile, and run) can help crystallize exactly what TVM is doing. I feel that making it object oriented instead somewhat obscures the steps. That said, I do agree that there are benefits to OOP such as discoverability and would be fine adding class methods if you feel strongly that its better. @jwfromm I got your point, which is also fair enough. Although I personally prefer the class method approach still, I don't feel it is strongly better than the proposed one, so I'm fine with it if others agree. -- 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: [email protected]
